mlflow-export-import icon indicating copy to clipboard operation
mlflow-export-import copied to clipboard

Add all entrypoints to a single Click Group

Open juftin opened this issue 1 year ago • 3 comments

Currently this package uses a number of entrypoints to access different functionality. Moving all of these Click Commands under a Singular Click group would be much easier.

I've already performed this work on an old fork of https://github.com/amesar/mlflow-export-import/. I'll open a PR on this repo instead.

Here's what it would look like:

❯ mlflow-export-import --help
Usage: mlflow-export-import [OPTIONS] COMMAND [ARGS]...

  MLflow Export / Import CLI: Command Line Interface

Options:
  --version  Show the version and exit.
  --help     Show this message and exit.

Commands:
  export-all          Export the entire tracking server All registered...
  export-experiment   Exports an experiment to a directory.
  export-experiments  Exports experiments to a directory.
  export-model        Export a registered model and all the experiment runs...
  export-models       Exports models and their versions' backing Run along...
  export-run          Exports a run to a directory.
  find-artifacts      Find artifacts that match a filename
  http-client         Interact with the MLflow Export/Import HTTP Client
  import-experiment   Import an experiment from a directory.
  import-experiments  Import a list of experiment from a directory.
  import-model        Import a registered model and all the experiment runs...
  import-models       Imports models and their experiments and runs.
  import-run          Imports a run from a directory.
  list-models         Lists all registered models.

juftin avatar Jul 26 '22 21:07 juftin

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?

amesar avatar Jul 27 '22 01:07 amesar

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?

Yup, here's a PR opened from my fork: https://github.com/mlflow/mlflow-export-import/pull/30

juftin avatar Jul 27 '22 01:07 juftin

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

amesar avatar Jul 27 '22 05:07 amesar