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

Add Mlflow tracking #18

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add Mlflow tracking #18

wants to merge 4 commits into from

Conversation

andodet
Copy link

@andodet andodet commented Feb 4, 2021

Summary

As mentioned in #17, adding training run and experiment tracking capabilities using Mlflow for the following templates:

  • templates/Image classification_PyTorch
  • templates/Image classification_scikit-learn

Details

This PR implements basic metric/model info logging features. As a second logical next step, model artifact logging (checkpoints and final model) will be added.

Checklist

  • all tests are passing (see README.md on how to run tests)
  • if you created a new template: it contains a file test-inputs.yml, which specifies a few input values to test the code template (the test is then automatically run by pytest)
  • you formatted all code with black
  • you checked all new functionality live, i.e. in the running web app
  • any generated code is formatted nicely, both in .py and in .ipynb ("nicely" = comparable to the existing templates)
  • you added comments in your code that explain what it does
  • the PR explains in detail what's new

This will:
* Add Mlflow code snippet to Jinja template.
* Add option in dropdown menu supporting user-provided experiment name.
* Add Mlflow option to test inputs.
This will:
* Add Mlflow code snippet to Jinja template.
* Add option in dropdown menu supporting user-provided experiment name.
* Add Mlflow option to test inputs.
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.

None yet

1 participant