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

Convention for specifying dependencies #1

Open
jsh9 opened this issue Feb 16, 2024 · 2 comments
Open

Convention for specifying dependencies #1

jsh9 opened this issue Feb 16, 2024 · 2 comments

Comments

@jsh9
Copy link

jsh9 commented Feb 16, 2024

The convention for specifying dependencies is usually a requirements.txt file like this:

pytorch
torchvision
torchaudio
pytorch-cuda==11.8
pandas
...

And then let people install them by pip install -r requirements.txt.

Conda may not be necessary, pip should be sufficient.

Also, it seems torchvision and torchaudio are not needed in the algorithm? There's no need to specify unused dependencies in the requirements.txt file.

@yzshi5
Copy link
Contributor

yzshi5 commented Feb 19, 2024

Hi Jian, I agree with that a requirements.txt may be necessary and more formal. Since this work is a pilot study, and has several follow ups. I will be more careful in further code release.

Usually, build a separate conda environment is a convenient way for testing a new model, especially for people without a good background in computer system. For the installation of unused torchvision and torchaudio, I just copied the command line for installing pytorch from the official website.

@jsh9
Copy link
Author

jsh9 commented Feb 20, 2024

Usually, build a separate conda environment is a convenient way for testing a new model, especially for people without a good background in computer system.

Yeah, people can just do pip install -r requirements.txt after they create a fresh conda environment.

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

2 participants