Make openmim available on conda-forge about mim HOT 22 CLOSED

openmim and model-index have been published on conda-forge.
https://anaconda.org/conda-forge/openmim

@zhouzaida Would you be willing to be listed as a maintainer of the package on conda-forge?
https://github.com/conda-forge/openmim-feedstock

from mim.

It seems model-index is not heavily used by openmin. The load() function is used once, and ModelIndex is used twice. I am not sure if it makes sense to make model-index an optional package for openmin since model-index is not actively maintained. Basically, put the modelindex import statement within the function rather than at the top-level import. In that way, it does not need to be listed as a dependency for PyPI. This can be a backup plan in case we can't get modelindex on conda-forge.

@zhouzaida Is it something you would consider?

https://github.com/open-mmlab/mim/blob/main/mim/commands/search.py#L295

from modelindex.load_model_index import load
from modelindex.models.ModelIndex import ModelIndex

from mim.

@zhouzaida Understood. Let's get model-index published first.

from mim.

I fully agree! If any help is needed I'm here. Just a reminder that you only need to publish once, because the updates are automatically done. It's worth entering the conda-forge ecosystem.

from mim.

Hi @giswqs and @arturdaraujo, thanks for your suggestion. It sounds good and we will make openmim available ASAP.

from mim.

We don't mind publishing and we can add you as maintainer. @giswqs is an experienced conda-forge publisher

from mim.

I just checked. openmin requires model-index, so will need to make model-index available first.

from mim.

@zhouzaida is model-index an absolute requirement for openmim? Can it be made as an optional package?

from mim.

I think this is the one
pypi: https://pypi.org/project/model-index/
github: https://github.com/paperswithcode/model-index

from mim.

@zhouzaida is model-index an absolute requirement for openmim? Can it be made as an optional package?

model-index is a required package. Runtime requirements can be found at https://github.com/open-mmlab/mim/blob/main/requirements/install.txt

from mim.

Let's publish model-index

from mim.

And model-index requires modelindex, so we need modelindex as well. https://pypi.org/project/modelindex/

from mim.

modelindex is an alias for the model-index library. More details at https://pypi.org/project/modelindex/.

from mim.

It is indeed a rabbit hole. And modelindex recipe fails to create. We might be stuck.

from mim.

modelindex has no github page it seems

from mim.

What a useless thing is "alias package". Does it make sense to simply remove modelindex from the recipe? I think we can talk to the devs from conda-forge

from mim.

Another alternative would be to ask model-index maintainers to remove modelindex, but they don't see to be active for a while

from mim.

I agree with you. What a weird way to list an alias package of itself as a dependency, but that alias package is not being maintained.

from mim.

This looks good! @zhouzaida

from mim.

It seems model-index is not heavily used by openmin. The load() function is used once, and ModelIndex is used twice. I am not sure if it makes sense to make model-index an optional package for openmin since model-index is not actively maintained. Basically, put the modelindex import statement within the function rather than at the top-level import. In that way, it does not need to be listed as a dependency for PyPI. This can be a backup plan in case we can't get modelindex on conda-forge.

@zhouzaida Is it something you would consider?

https://github.com/open-mmlab/mim/blob/main/mim/commands/search.py#L295

from modelindex.load_model_index import load
from modelindex.models.ModelIndex import ModelIndex

This command mim search is used frequently and it would be user-friendly to install model-index automatically when installing openmim.

from mim.

I'm also looking for alternatives that are up to date and maintained by a larger organization.

This one may work?

• https://github.com/google/ml-metadata

from mim.

openmim and model-index have been published on conda-forge. https://anaconda.org/conda-forge/openmim

@zhouzaida Would you be willing to be listed as a maintainer of the package on conda-forge? https://github.com/conda-forge/openmim-feedstock

Yes, I would. Thanks for your contributions.

from mim.