Andre Mesarovic comments

Results 37 comments of


                                            Andre Mesarovic

Use pre-commit to standardize code formatting

Sounds interesting, let me check it out. Only issue might be that Databricks notebooks are saved as ".py" files, so we'd have to exclude them somehow.

Use pre-commit to standardize code formatting

We might also want to check out how the core mlflow repo handles code formatting so as to be compatible with them.

importing models with mlflow-artifacts: source

Good catch. I'm finishing up on another PR now. Will merge yours after that.

importing models with mlflow-artifacts: source

Tests passed and I approved the change.

Add all entrypoints to a single Click Group

Sounds great. I've thought of doing this too, similar to the mlflow CLI, but time sort of ran out. Do you have a link to your fork?

Add all entrypoints to a single Click Group

I am on PTO until August 14, so responses might be delayed. :)

Add start_date and end_date in importer for runs

end_time is a strictly read-only value set by the system. start_time is also set by the system with an option in create_run() to pass in a time though w/o being...

Incorrect value into mlflow_http_client

MLFLOW_TRACKING_URI has to be set. The tool does not handle file-based store where not MLFLOW_TRACKING_URI is needed. Not sure what the issue is.

Can't import-run in single mode

Hey there, sorry for the delay. > where the "same" experiment exists already (name=NAME, ID=YYY). Is that an intended use case? Yes, this should be OK. If experiment exists import...

MLFlow Exception - Databricks Personal Access Token timeout

When you create a PAT you specify its lifetime in days. Not aware of a problem for long-running requests. Best to ask in the https://community.databricks.com/s/