GEM-benchmark / NL-Augmenter

NL-Augmenter 🦎 → 🐍 A Collaborative Repository of Natural Language Transformations

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

issue while build

pawan2411 opened this issue · comments

not sure why module is failing while building....i am using direct install for one library from git (see first line of requirements.txt)
it is working in local and also on multiple GCS vms (where i did dev).
reference:
https://github.com/GEM-benchmark/NL-Augmenter/pull/113/checks?check_run_id=3040556102

Hi @pawan2411 : The build is failing because you're using invalid format for wptools installation in the requirements.txt file for your transformation. This is the original library: https://github.com/siznax/wptools. Use wptools==version instead of using your own fork of the library!

@AbinayaM02 because original wptools was using pycurl which is not supported in RHEL by default(i suppose RHEL is the one which is used to build), hence i changed wptools code, for this i also created a PR in wptools which was merged(siznax/wptools#168), but I am not sure when will wptools officially update and release new version, so I have to give git link in requirements instead ( https://stackoverflow.com/questions/16584552/how-to-state-in-requirements-txt-a-direct-github-source ), also when i do "pip install -r requirements" this does work for me, not sure why this requirements file is not working in the build machine.

looks like updating setuptools will sort this out, did that in new commit...waiting for the next build @AbinayaM02

@pawan2411 : Setup.py has an argument called install_requires which doesn't accept the git links as is. That's the error thrown doing build. setuptools is already installed in the initial requirement in the github actions runner. So, not sure if the build will pass!
Let's wait and see.

@AbinayaM02 it failed again! let me check if we can do any workaround.

@AbinayaM02 added "wptools @ git+git://github.com/pawan2411/wptools@30738f1" line instead....
reference : https://stackoverflow.com/questions/26061610/pip-install-dependency-links
can u again start the build pls

@AbinayaM02 above fix worked! Closing the issue. Thanks