Skip to content

Kf1.9 common lib components update #340

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

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

mathis-marcotte
Copy link
Contributor

for https://jirab.statcan.ca/browse/BTIS-661

Just for what is found under /kubeflow-common-lib/projects/kubeflow/src/lib

@mathis-marcotte
Copy link
Contributor Author

mathis-marcotte commented Feb 25, 2025

Failing build will be fixed with this pr #339
Failing because it requires code changes that are outside the scope of this ticket.

Failing tests will be fixed with ticket https://jirab.statcan.ca/browse/BTIS-673

Copy link
Contributor

@wg102 wg102 left a comment

Choose a reason for hiding this comment

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

I will consider this review partial until I run the code and see visual changes to make sure it works as expected. Unless it is not part of the scope

Copy link
Contributor

@wg102 wg102 left a comment

Choose a reason for hiding this comment

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

Approving PR since deadlock with the other one. Any test or things looking weird shall be found and decided when UI is up and running and able to see how it affects user experience

@mathis-marcotte mathis-marcotte merged commit dec6dbc into KF1.9-jupyter-apis Feb 26, 2025
1 of 6 checks passed
@mathis-marcotte mathis-marcotte deleted the kf1.9-common-lib branch February 26, 2025 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants