Skip to content
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

requirements.txt missing library #22

Open
yassineAlouini opened this issue May 26, 2023 · 3 comments
Open

requirements.txt missing library #22

yassineAlouini opened this issue May 26, 2023 · 3 comments

Comments

@yassineAlouini
Copy link

yassineAlouini commented May 26, 2023

It seems that the requirements.txt file is missing some of the necessary libraries. Besides, some aren't necessary (such as os and warnings, which come in the base library).

I can contribute an MR that adds these. 👌

@yassineAlouini
Copy link
Author

The following requirements seem to work:

matplotlib
tqdm
numpy
opencv-python
torch
torchvision

@Skwarson96
Copy link

I already did PR #18

@AustinStarnes
Copy link

AustinStarnes commented Jul 24, 2024

To be clear, os and warnings in the requirements file isn't just "unnecessary", but actually break the installation instruction pip install -r requirements.txt which was provided in the readme, since these modules are not provided by pip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants