Skip to content

Fix the Model Validation#131

Merged
arpitjasa-db merged 3 commits into
databricks:mainfrom
qili86:main
Jan 10, 2024
Merged

Fix the Model Validation#131
arpitjasa-db merged 3 commits into
databricks:mainfrom
qili86:main

Conversation

@qili86

@qili86 qili86 commented Jan 8, 2024

Copy link
Copy Markdown
Contributor

Current default template for Model Validation part fails because of the two errors below:

model validation fail

error_1. uc

The full error msg is below:
Validation failed : API request to list files under path `` failed with status code 400. Response body: {"error_code":"INVALID_PARAMETER_VALUE","message":"Invalid name 'local_dev.custom_project.custom_project_model'. A workspace registered model name must be a non-empty UTF-8 string and cannot contain forward slashes(/), periods(.), or colons(:). If trying to access models in the Unity Catalog, make sure to install version 2.4.1 or above of the MLflow Python client, then configure the MLflow client to access models in Unity Catalog via mlflow.set_registry_uri('databricks-uc') before accessing models."}

Fix to error_1: add "mlflow.set_registry_uri('databricks-uc')"

error_2: targets

Fix to error_2: change to "fare_amount" which is the correct prediction target

@arpitjasa-db arpitjasa-db left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the changes @qili86 ! Would you mind generating a Stacks repo from your local branch and adding screenshots of the validation workflow working as well?

@qili86

qili86 commented Jan 10, 2024

Copy link
Copy Markdown
Contributor Author

Thanks for the changes @qili86 ! Would you mind generating a Stacks repo from your local branch and adding screenshots of the validation workflow working as well?

@arpitjasa-db I generated the stack repo "test_mv" from my local branch, and the model validation part is now working fine without error
Screenshot 2024-01-09 at 9 16 10 PM

@arpitjasa-db arpitjasa-db left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @qili86 !

@arpitjasa-db
arpitjasa-db merged commit cd33ef6 into databricks:main Jan 10, 2024
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