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

README edit to change Apple Silicon install instructions (to fix a break introduced by pytorch 2) #8122

Merged
merged 2 commits into from
Jan 4, 2024

Conversation

stephenmcconnachie
Copy link
Contributor

@stephenmcconnachie stephenmcconnachie commented Jan 4, 2024

What does this PR do ?

This PR aims to correct a small error in the Apple Silcon installation instructions in README, where a triton dependency is problematic due to pytorch 2. The use of /reinstall.sh doesn't work, so using pip install instead. Also added a note about MacBook, to clarify that only asr toolkit is supported.

Additional Information

Changed Apple Silcon install instrictions based on issue 8116: NVIDIA#8116 (comment)

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>
Added comment 

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>
Copy link
Collaborator

@titu1994 titu1994 left a comment

Choose a reason for hiding this comment

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

Thanks !

@titu1994 titu1994 merged commit dae28da into NVIDIA:main Jan 4, 2024
4 checks passed
minitu pushed a commit to minitu/NeMo that referenced this pull request Jan 19, 2024
…eak introduced by pytorch 2) (NVIDIA#8122)

* Update README.rst

Changed Apple Silcon install instrictions based on issue 8116: NVIDIA#8116 (comment)

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>

* Update README.rst

Added comment 

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>

---------

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>
@TomTom101
Copy link

git clone and cd'ing into NeMo became obsolete when installing from pypi. Imo those commands can be removed?

git clone https://github.com/NVIDIA/NeMo
cd NeMo

In my case, I only get v1.21 installed from pypi. With pip install 'nemo-toolkit[asr]==1.22.0' I again run into an triton dependency issue:
ERROR: Could not find a version that satisfies the requirement triton (from nemo-toolkit) (from versions: none)

@TomTom101
Copy link

TomTom101 commented Feb 10, 2024

According to this comment, macOS is not supported by triton at all.
What eventually did work for me was removing triton from the requirements altogether and install from the local checkout.
$ pip install '.[asr]'

ssh-meister pushed a commit to ssh-meister/NeMo that referenced this pull request Feb 15, 2024
…eak introduced by pytorch 2) (NVIDIA#8122)

* Update README.rst

Changed Apple Silcon install instrictions based on issue 8116: NVIDIA#8116 (comment)

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>

* Update README.rst

Added comment 

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>

---------

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>
Signed-off-by: Sasha Meister <ameister@nvidia.com>
rohitrango pushed a commit to rohitrango/NeMo that referenced this pull request Jun 25, 2024
…eak introduced by pytorch 2) (NVIDIA#8122)

* Update README.rst

Changed Apple Silcon install instrictions based on issue 8116: NVIDIA#8116 (comment)

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>

* Update README.rst

Added comment 

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>

---------

Signed-off-by: Stephen <stephen.mcconnachie@bfi.org.uk>
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