You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In Python it is a valid convention to use None as a default value for a parameter, however for Python components if we want to document in the deployment manifest all parameters with a default value of None, currently we would have to pass is as a STRING parameter and do post-processing in the Python wrapper (e.g. explicitly checking if the STRING is "None" and if so setting the parameter to None), this would introduces another convention to document.
The text was updated successfully, but these errors were encountered:
* Remove duplicated pipeline metrics
The inference request count and latency are tracked in the corresponding histogram,
under the '_count' and '_sum' metrics respectively.
Duplicating these adds almost 40% extra timeseries pressure per pipeline
and would likely confuse users who can't be sure which metric they should use.
* Add explanatory comment on aggregate metrics for pipelines
* Split gateway metrics constants into blocks & tidy surrounding comments
* Split agent metrics constants into blocks & tidy surrounding comments
* Remove duplicated agent metrics
The same reasoning applies as for the duplicated pipeline metrics.
* Make pipeline gateway metric names private as usage is purely package-internal
* Remove deprecated metrics from agent metrics test
* Make agent metric names private
* Update metrics in sample notebook to use histogram-provided ones
* Update generated Markdown version of updated notebook
* Update Markdown files generated from samples notebooks
This is the remaining changes from running 'make -C samples/ all'.
It consists of whitespace changes and some language indicators for cells.
In Python it is a valid convention to use
None
as a default value for a parameter, however for Python components if we want to document in the deployment manifest all parameters with a default value ofNone
, currently we would have to pass is as a STRING parameter and do post-processing in the Python wrapper (e.g. explicitly checking if the STRING is "None" and if so setting the parameter toNone
), this would introduces another convention to document.The text was updated successfully, but these errors were encountered: