-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
requiremtns.txt needs to be updated #295
Comments
I am not sure if everything in my virtual environment is quite necessary for running the framework. We have to check this with a clean new installation of an environment and by resolving the dependencies one by one and then freezing that environment as the minimum requirements. |
Suggestion: try to keep only necessary ones for the framework in the root |
@AdmiralDarius Do you have a version with the minimum requirement setting or is anyone looking to make this right now? |
@AdmiralDarius please explain your issue with the torch versions upper than |
On the most recent version of develop_newLC i tried torch 1.9.1 and torch 1.10 and the problem is resolved |
but on the main branch if i run wiqa_aug.py with torch 1.9.1 i get this: ModuleNotFoundError Traceback (most recent call last) 5 frames /content/DomiKnowS/regr/program/program.py in () /content/DomiKnowS/regr/sensor/init.py in () /content/DomiKnowS/regr/sensor/sensor.py in () /content/DomiKnowS/regr/graph/init.py in () /content/DomiKnowS/regr/graph/dataNode.py in () ModuleNotFoundError: No module named 'torch.tensor' |
@hossein
put the important stuff from your virtual environment into the requriements.txt with their version on git. like this:
![image](https://user-images.githubusercontent.com/39480854/136292719-82610f56-8c14-4952-8cf1-f2f8182ce9fc.png)
also add spacy , networkx , numpy , request, panda and tqdm
The text was updated successfully, but these errors were encountered: