Skip to content
This repository has been archived by the owner on Feb 24, 2022. It is now read-only.

Please add tensorflow, keras, pytorch, etc. to the requirements.txt file #5

Open
sbbogdan opened this issue Mar 8, 2018 · 3 comments
Assignees
Labels
wontfix This will not be worked on

Comments

@sbbogdan
Copy link

sbbogdan commented Mar 8, 2018

For the deep learning nanodegree we are going to have to use at least one, or multiple of these libraries. It makes no sense for their presence not to be in the requirements.txt file, or even mentioned anywhere in the instructions.

It would be nice if you could:
a) Add an README.md step beneath 'Python Packages' that explicitly requires students to open the requirements.txt file
b) Have commented out lines for popular packages like tensorflow, keras, pytorch, etc. and ask students to review what they think they will use, and then uncomment such out prior to proceeding. Realistically, hard drive space is so cheap and plentiful these days that these libraries should not be a significant space burden and it is probably just easier to include at least keras, pandas, and tensorflow by default. Students can uninstall what they don't need.

@ablacklama
Copy link

ablacklama commented Mar 9, 2018

pandas/h5py too.

@sbbogdan
Copy link
Author

sbbogdan commented Mar 9, 2018

+1 to pandas and h5py. Those were needed for this project.

Tensorflow can be installed with pip, that's what I did. It just won't be GPU accelerated AFAIK.

@ablacklama
Copy link

you're completely right. I honestly forget that cpu tensorflow even exists sometimes... I removed it from my comment

@abhiojha8 abhiojha8 self-assigned this Mar 4, 2021
@abhiojha8 abhiojha8 added the wontfix This will not be worked on label Mar 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants