googleapis / googleapis/python-aiplatform
fps option is missing in VideoMetadata
- Dominant language
- Python
- Stars
- 905
- Forks
- 465
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 44
Description
FPS option is missed in `class VideoMetadata(proto.Message)` in types/content.py
It should exist according to both docs (v1 and v1beta1):
https://cloud.google.com/vertex-ai/generative-ai/docs/reference/rest/v1/Content
https://cloud.google.com/vertex-ai/generative-ai/docs/reference/rest/v1beta1/Content
And also is existent in VideoMetadata in `content.proto`:
https://github.com/googleapis/googleapis/blob/97763d6efb2d35474e0363c254ee6994837a08fa/google/ai/generativelanguage/v1/content.proto#L107
as:
```
message VideoMetadata {
// Optional. The start offset of the video.
google.protobuf.Duration start_offset = 1
[(google.api.field_behavior) = OPTIONAL];
// Optional. The end offset of the video.
google.protobuf.Duration end_offset = 2
[(google.api.field_behavior) = OPTIONAL];
// Optional. The frame rate of the video sent to the model. If not specified,
// the default value will be 1.0. The fps range is (0.0, 24.0].
double fps = 3 [(google.api.field_behavior) = OPTIONAL];
}
```
Contributor guide
Research direction
Open types/content.py and inspect the VideoMetadata class, then compare it with the linked content.proto definition and the v1/v1beta1 documentation. Done means VideoMetadata exposes the documented fps option with the stated default and range, matching the proto definition.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api, machine-learning
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 55/100