Skip to content
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

Use MLFLOW_TRACKING_URI environment variable by default instead of mlruns #321

Closed
Galileo-Galilei opened this issue May 24, 2022 · 0 comments · Fixed by #325
Closed

Use MLFLOW_TRACKING_URI environment variable by default instead of mlruns #321

Galileo-Galilei opened this issue May 24, 2022 · 0 comments · Fixed by #325
Assignees
Labels
enhancement New feature or request

Comments

@Galileo-Galilei
Copy link
Owner

Description

It is common to specify the mlflow tracking uri through the MLFLOW_TRACKING_URI environment variable. We should have a way to use this variable as the default one in kedro-mlflow

Context

When using kedro-mlflow in a managed cluster where the tracking uri is defined dynamically, I can't use a fixed uri in the mlflow.yml config file.

Possible Implementation

When server.mlflow_tracking_uri is null in mlflow.yml, use MLFLOW_TRACKING_URI instead of mlruns as a default.

mlflow_tracking_uri: str = "mlruns"

@Galileo-Galilei Galileo-Galilei self-assigned this May 24, 2022
@Galileo-Galilei Galileo-Galilei added the enhancement New feature or request label May 24, 2022
Galileo-Galilei added a commit that referenced this issue May 30, 2022
Galileo-Galilei added a commit that referenced this issue May 31, 2022
Galileo-Galilei added a commit that referenced this issue May 31, 2022

Partially verified

This commit was created on github.com and signed with GitHub’s verified signature.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
…g uri in configuration (#321)
Galileo-Galilei added a commit that referenced this issue May 31, 2022

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
…g uri in configuration (#321)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

1 participant