Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add parameters_value in PipelineJob for schema > 2.0.0 #817

Merged
merged 14 commits into from Nov 15, 2021

Conversation

ji-yaqi
Copy link
Contributor

@ji-yaqi ji-yaqi commented Nov 3, 2021

No description provided.

@product-auto-label product-auto-label bot added the api: aiplatform Issues related to the AI Platform API. label Nov 3, 2021
@ji-yaqi ji-yaqi requested a review from chensun November 3, 2021 22:37
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 3, 2021
@ji-yaqi ji-yaqi force-pushed the protobuf_val branch 2 times, most recently from bcad4c5 to 6c689d2 Compare November 4, 2021 21:08
@ji-yaqi ji-yaqi added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 9, 2021
@ji-yaqi
Copy link
Contributor Author

ji-yaqi commented Nov 9, 2021

Hold the PR for now, for testing with KFP 2.0

Copy link
Member

@chensun chensun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thanks!

@@ -122,7 +138,7 @@ def build(self) -> Dict[str, Any]:

def _get_vertex_value(
self, name: str, value: Union[int, float, str, bool, list, dict]
) -> Dict[str, Any]:
) -> Union[Dict[str, Any], int, float, str, bool, list, dict]:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: maybe remove Dict[str, Any] since it's covered by dict?

@ji-yaqi ji-yaqi removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 15, 2021
@ji-yaqi ji-yaqi merged commit 900a449 into googleapis:main Nov 15, 2021
@ji-yaqi ji-yaqi deleted the protobuf_val branch November 15, 2021 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: aiplatform Issues related to the AI Platform API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants