Add requirements.txt file #1
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
Many thanks. Could you please only pin versions where it's required? In general we try to have things run with latest version at all times, not pinned versions. Also could you please revert your change to the notebook, since on your own env that cell is only required once, not every time you run it |
requirements.txt
Outdated
| huggingface-hub==0.10.1 | ||
| transformers==4.23.0 | ||
| fastcore==1.5.27 | ||
| fastdownload==0.0.7 | ||
| git+https://github.com/huggingface/diffusers.git@main#egg=diffusers |
There was a problem hiding this comment.
Thanks for this! Just a thought:
This requirements.txt depends on what you are starting with inside your environment.
For example, many things already come on Google Colab (scipy, matplotlib, etc.).
But if someone is setting up a fresh env (Docker, Conda, etc.), then they will need
to install other dependencies like scipy, matplotlib, Jupyter and so on.
As I was setting up a fresh Docker file this evening in order to play
with the first notebook, stable_diffusion, I found I needed to install other packages other than those listed here.
There was a problem hiding this comment.
I removed the entries for fastcore and fastdownload and added one for fastai since it brings both fastcore and fastdownload. Moreover, it brings other necessary libraries, like torch, scipy, and matplotlib. I don't think that installing notebook or jupyterlab is part of this requirements.txt file. You need Jupyter just to be able to open the ipynb file, and you can start from a Jupyter-provided Docker image for this. But, I'm happy to discuss it.
requirements.txt
Outdated
| transformers==4.23.0 | ||
| fastcore==1.5.27 | ||
| fastdownload==0.0.7 | ||
| git+https://github.com/huggingface/diffusers.git@main#egg=diffusers |
There was a problem hiding this comment.
| git+https://github.com/huggingface/diffusers.git@main#egg=diffusers | |
| diffusers |
Version 0.4.2 was released a few hours ago, it should work out of the box now.
There was a problem hiding this comment.
That's great. I made the change.
Add a 'requirements.txt' file to install the dependencies needed to run the 'stable_diffusion.ipynb' file. Signed-off-by: Dimitris Poulopoulos <dimpo@arrikto.com>
|
@jph00 I made the changes you asked for and added some more to address the comments by @DrChrisLevy and @pcuenca. I also edited the commit message since we do not install |
Add a
requirements.txtfile to install the dependencies needed to run thestable_diffusion.ipynbfile.Install the
diffuserslibrary from source because of commit92d70863663662669ee3c376909be1f876e00965which is needed for fp16 to work.Refs huggingface/diffusers#769
Signed-off-by: Dimitris Poulopoulos dimpo@arrikto.com