-
Notifications
You must be signed in to change notification settings - Fork 130
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
Mlflow update 2209 #367
Mlflow update 2209 #367
Conversation
ok to test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@pdmack pr changes look good but noticed a couple other updates we need to add:
should i create a separate pr for these or do you want to add this pr? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Weird port bug with mlflow 1.29.0
Somewhat better version spec
…mlflow-update-2209
@gpucibot merge |
Some changes in support of production DFP and also image hygiene