Skip to content

Fix requirements#18

Open
Skwarson96 wants to merge 3 commits intoZrrSkywalker:mainfrom
Skwarson96:fix_requirements
Open

Fix requirements#18
Skwarson96 wants to merge 3 commits intoZrrSkywalker:mainfrom
Skwarson96:fix_requirements

Conversation

@Skwarson96
Copy link
Copy Markdown

@ZrrSkywalker Hi! I fix requitements.txt file, because when you want to install the os and warnings packages with pip, there are errors. Os and warnings do not require additional installation. Additionally, I added pytorch and torchvision libraries.

I have also added to the readme instructions on how to create an environment in a virtual environment

@Skwarson96
Copy link
Copy Markdown
Author

Why did you remove torch? Without installing torch after run: python persam.py you get an error: ModuleNotFoundError: No module named 'torch'

@ArenaGrenade
Copy link
Copy Markdown

ArenaGrenade commented Apr 23, 2024

The environment can be conda-based or pip-based and removing pytorch out to allow users to choose how they want to install it - either with conda or pip should be the way they do it.

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

Successfully merging this pull request may close these issues.

3 participants