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

kedro mlflow ui does not use arguments from mlflow.yml #187

Closed
Galileo-Galilei opened this issue Apr 5, 2021 · 0 comments · Fixed by #189
Closed

kedro mlflow ui does not use arguments from mlflow.yml #187

Galileo-Galilei opened this issue Apr 5, 2021 · 0 comments · Fixed by #189
Assignees
Labels
bug Something isn't working
Milestone

Comments

@Galileo-Galilei
Copy link
Owner

Description

As described in this stackoverflow question, the ui command does not use the options

Context & Steps to Reproduce

  • Create a kedro project
  • Call kedro mlflow init
  • Modify the port in mlflow.yml to 5001
  • Launch kedro mlflow ui

Expected Result

The mlflow UI should open in port 5001.

Actual Result

It opens on port 5000 (the default).

Your Environment

Include as many relevant details about the environment in which you experienced the bug:

  • kedro version: 0.17.0
  • kedro-mlflow version: 0.6.0
  • Python version used (python -V): 3.6.8
  • Operating system and version: Windows

Does the bug also happen with the last version on master?

Yes

Solution

We should pass the arguments in the command:

subprocess.call(
["mlflow", "ui", "--backend-store-uri", mlflow_conf.mlflow_tracking_uri]
)

@Galileo-Galilei Galileo-Galilei added the bug Something isn't working label Apr 5, 2021
@Galileo-Galilei Galileo-Galilei self-assigned this Apr 5, 2021
@Galileo-Galilei Galileo-Galilei changed the title kedro mlflow ui does not use argumetns from mlflow.yml kedro mlflow ui does not use arguments from mlflow.yml Apr 6, 2021
Galileo-Galilei added a commit that referenced this issue Apr 9, 2021
…e ui command and enable runtime overwriting
Galileo-Galilei added a commit that referenced this issue Apr 9, 2021
…e ui command and enable runtime overwriting
@Galileo-Galilei Galileo-Galilei added this to the Release 0.7.1 milestone Apr 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
1 participant