Detect if MLFlow Model was updated#4207
Conversation
Test Results for assets-test0 tests 0 ✅ 0s ⏱️ Results for commit de114a3. ♻️ This comment has been updated with latest results. |
Test Results for scripts-test79 tests 79 ✅ 8m 16s ⏱️ Results for commit de114a3. ♻️ This comment has been updated with latest results. |
This reverts commit e732662.
…lflow-model-tag-req
…ml-assets into mlflow-model-tag-req
This reverts commit 0700e44.
sophies927
left a comment
There was a problem hiding this comment.
This looks good to me from a Foundry Local perspective! Just to confirm that I'm fully understanding the changes, this will only cause an error if MLFlow model is updated, but it looks like nothing will happen if a custom model variant with an MLFlow parent model (e.g., some of the Phi models) is updated, correct? And are there any plans to fully delete/remove MLFlow models entirely?
If no to both, local models will be unaffected since we only update/publish new versions for model variants.
Correct, no impact for Foundry Local since those are all custom. We're still looking into what to do with existing MLFlow models, but right now just latest/new ones will be impacted by this gate update. You might've been added as a reviewer since I was ensuring a custom model wasn't impacted (and happened to choose a Foundry Local one), thanks for taking a look! :) |
No description provided.