googleapis / googleapis/python-aiplatform

Logging time series metrics to an experiment run with step=0 goes the wrong way

未关闭
#2,133 3 条评论 1 个 reaction 已指派 0 人 在 GitHub 查看
api: vertex-ai
主要语言
Python
星标
905
派生
465
平均合并
1 天 13 小时
30 天内合并 PR
44

描述

#### Environment details

- OS type and version: Linux
- Python version: `all`
- pip version: `all`
- `google-cloud-aiplatform` version: `main`

#### Steps to reproduce

1. call `ExperimentRun.log_time_series_metrics()` with `step=0`.

The main issue here is the condition at [this line](https://github.com/googleapis/python-aiplatform/blob/main/google/cloud/aiplatform/metadata/experiment_run_resource.py#L955):

`if not step:`

Should be:

`if step is None:`

A caller that counts its steps starting at 0, would unnecessarily start a search for the latest time series step and that could trigger other issues, for example, hitting a limit on the size of the time series to read, which is not relevant here, but could be avoided if this condition was testing the right value.

#### Code example

```python
run = aiplatform.ExperimentRun(experiment=my_experiment, run_name=my_run_name)
run.log_time_series_metrics(my_metrics, step=0)
```

#### Stack trace
![Screen Shot 2023-04-25 at 12 31 47 PM](https://user-images.githubusercontent.com/1761609/234343653-f7cf5412-24f8-4a27-b43a-9706744ba1c2.png)

贡献指南

打开贡献指南

调研方向

Start at google/cloud/aiplatform/metadata/experiment_run_resource.py around line 955 and trace ExperimentRun.log_time_series_metrics(). Verify that the provided step=0 follows the explicit-step path rather than the missing-step path; done when logging at step 0 no longer searches for the latest time-series step.

由索引模型根据 Issue 内容生成。

评估

技术栈
google-cloud, python
领域
api, machine-learning
Issue 类型
缺陷
难度
1/5
预计耗时
1-3 小时
活跃度
停滞
描述清晰度
描述清楚
新手友好度
52/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。