Closed ptynecki closed 2 years ago
having the same issue right now looking for a solution. (i am not on ESM team)
@brycejoh16
PR is waiting: https://github.com/facebookresearch/esm/pull/254
@ptynecki thank you!
@tomsercu PR #254 is resolving the issue problem but there is another during the installation.
The fairscale
package is required (not added to setup.py as well).
Hi @ptynecki @brycejoh16 thanks for spotting that! It's fixed now in https://github.com/facebookresearch/esm/pull/254, the fairscale dependency will be fixed soon.
I might be wrong, but the setup.py is rather curiously written, I don't know your guys setup and I have not looked at the history, but it very much looks like be a case of files being erroneously switched for a test or similar. Say
setuptools.setup
function lacks a requirement argument (i.e. no fairscale module))setuptools.find_packages
fills subpackages without issues (okay, PyCharm does it but is bad)__version__
is read very oddlySo might be worth checking there's no weird accident in the version!
Thanks yes much about our build/release process could be improved.
@ptynecki let us know if your issues are resolved now. Thanks all for your input!
Bug description "model" directory is missed in "packages" argument in setup.py script. That issue raised
ModuleNotFoundError: No module named 'esm.model'
exception.Reproduction steps Try to install
esm
package from PyPI or from the GitHub repository.Expected behavior I was expected successful installation.
Logs