databricks / mlops-stacks

This repo provides a customizable stack for starting new ML projects on Databricks that follow production best-practices out of the box.
https://docs.databricks.com/en/dev-tools/bundles/mlops-stacks.html
Apache License 2.0
416 stars 141 forks source link

minor change to deploy.py.tmpl to fix the duplicated alias assigned to the created model in UC #160

Closed yinxi-db closed 1 month ago

yinxi-db commented 2 months ago

change all alias to lower case, mlflow UC model registry does not different upper/lower case in alias. The original code leads to model in UC have both alias "champion" and "challenger" image

arpitjasa-db commented 2 months ago

@yinxi-db bit confused, if aliases are case-insensitive, wouldn't this change be a no-op? Have we confirmed this works?

yinxi-db commented 2 months ago

@yinxi-db bit confused, if aliases are case-insensitive, wouldn't this change be a no-op? Have we confirmed this works?

yes, see the registered model here. Without the change, it will create a version with 2 alias as shown in the screenshot earlier and with the update, version 3 has the alias champion since it passed validation

arpitjasa-db commented 1 month ago

This was done as part of https://github.com/databricks/mlops-stacks/pull/165