Skip to content

Conversation

@yeshaokai
Copy link
Collaborator

@yeshaokai yeshaokai commented Jun 2, 2024

  • Removed modules that are not needed

  • Removed requirements that are not needed

  • Added installation script to enforce that one needs to install conda yaml before running pip install. This is because library like hdf5's dependency cannot be addressed by pip install. Therefore people run into hdf5 dependency missing after even after pip install

@yeshaokai yeshaokai requested a review from MMathisLab June 2, 2024 08:38
- AmadeusGPT is a Python package hosted on `pypi`. You can create a virtual env (conda, etc, see below*) or Docker and run:
```python
pip install 'amadeusgpt[streamlit]'
- We prepare installation script install.sh. Make sure you modify the bash script to fit your own.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why are you deleting the pypi package? it is on pypi ;)

Copy link
Collaborator Author

@yeshaokai yeshaokai Jun 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because I realize that, at least when I was working on it, one can't install deeplabcut with pip install alone. It needs to use the conda yaml to first install hdf5 and many users don't understand why that's the case. So inside the installation script, I force users to create a conda env via conda yaml and then pip install.

The installation script

source /Users/shaokaiye/miniforge3/bin/activate

conda env create -f conda/amadesuGPT.yml

conda activate amadeusgpt

pip install -e .[streamlit]

In colab, because hdf5 is pre-installed, then they will run into slightly different problem such as pytable vs. tables from deeplabcut. I overheard you guys might have solved it. Let me know if there is better way to solve it.

@yeshaokai yeshaokai closed this Jul 7, 2024
@yeshaokai yeshaokai deleted the shaokai/fix_install branch July 7, 2024 07:43
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