unionml icon indicating copy to clipboard operation
unionml copied to clipboard

[FEATURE] Add `conda` installation option for `unionml`

Open sugatoray opened this issue 3 years ago • 9 comments

It will be helpful to have a conda installation option. Although you could publish the library in your own channel at Anaconda, when other libraries depend on your library and they want to make theirs available on conda-forge channel, unionml will be necessary to have on conda-forge channel.

This could possibly facilitate more downstream adoption of unionml.


:tada: I have started the work of adding unionml to conda-forge channel already.

  • https://github.com/conda-forge/staged-recipes/pull/19349

Once it is added to conda-forge, you can install it with:

conda install -c conda-forge unionml

:point_right: Or, if you want a rather quicker installation, use mamba instead of conda for quick environment resolution.

# install mamba in your conda-environment (if not already installed)
conda install -c conda-forge mamba
# install unionml with mamba instead of conda (for quick installation)
mamba install -c conda-forge unionml

sugatoray avatar Jun 22 '22 10:06 sugatoray

Notes

Several prerequisites were necessary to take care of before this PR (https://github.com/conda-forge/staged-recipes/pull/19349) could be possible.

  • Add dependency mangum to conda-forge: https://github.com/conda-forge/staged-recipes/pull/19362

  • Update dependency versions on conda-forge for a few unionml dependencies:

    • Update flytekit from v0.26.1 to v1.0.5: https://github.com/conda-forge/flytekit-feedstock/pull/12
    • Update flyteidl from v0.22.0 to v1.1.6: https://github.com/conda-forge/flyteidl-feedstock/pull/12
    • Update grpcio-status from v1.41.1 to v1.46.3: https://github.com/conda-forge/grpcio-status-feedstock/pull/7

The PyPI source of unionml does not have any requirements.txt file at the moment. This causes problems with conda-forge packaging while installing from the source (.tar.gz file). I have temporarily circumvented this by dynamically creating the requirements.txt file. But this should be fixed at the source here.

  • Issue: #126
  • PR (to fix this issue): #125

sugatoray avatar Jun 22 '22 10:06 sugatoray

cc: @cosmicBboy

sugatoray avatar Jun 22 '22 11:06 sugatoray

UnionML is now available on conda-forge.

install_UnionML

sugatoray avatar Jun 27 '22 18:06 sugatoray

cc: @cosmicBboy

sugatoray avatar Jun 27 '22 18:06 sugatoray

fixed by https://github.com/conda-forge/staged-recipes/pull/19349

cosmicBboy avatar Jun 27 '22 18:06 cosmicBboy

Re-opening this... just need to add conda-forge install instructions in the README and installation section of the docs, then we can close this

cosmicBboy avatar Jun 27 '22 18:06 cosmicBboy

I had not checked earler, but there was a similar issue opened already.

  • Closes https://github.com/unionai-oss/unionml/issues/73

sugatoray avatar Jun 27 '22 19:06 sugatoray

@cosmicBboy Could you please assign this issue to me?

sugatoray avatar Jun 27 '22 19:06 sugatoray

@cosmicBboy Please feel free to close this issue whenever you find it appropriate.

sugatoray avatar Aug 01 '22 22:08 sugatoray

https://github.com/unionai-oss/unionml/issues/142 tracks the work to add instructions. Closing this ticket.

eapolinario avatar Sep 27 '22 00:09 eapolinario