-
Notifications
You must be signed in to change notification settings - Fork 607
[PyTorch][Core][JAX] Expand troubleshooting docs #2602
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
base: main
Are you sure you want to change the base?
[PyTorch][Core][JAX] Expand troubleshooting docs #2602
Conversation
Signed-off-by: Jeremy Berchtold <jberchtold@nvidia.com>
Greptile SummaryThis PR expands the troubleshooting documentation in README.rst by adding three new sections covering recently debugged installation issues:
The new content follows the existing documentation structure with symptom/solution pairs and provides actionable guidance for users experiencing these issues. The documentation additions are straightforward and helpful for users encountering these specific problems. Confidence Score: 4/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant README as README.rst
participant Docs as Troubleshooting Docs
User->>README: Encounters installation issue
README->>Docs: Check Troubleshooting section
alt UV/Virtual Environment Issue
Docs->>User: Check "Problems using UV or Virtual Environments"
User->>Docs: Review Import Error solution
User->>Docs: Review cuDNN Loading Failed solution
User->>Docs: Review Building Wheels solution
else JAX-specific Issue
Docs->>User: Check "JAX-specific Common Issues"
User->>Docs: Review FFI Issues solution
Note over User,Docs: Recommends --no-build-isolation flag
else Other Build Issue
Docs->>User: Check existing troubleshooting sections
User->>Docs: Review compiler, headers, or build resource issues
end
User->>User: Apply solution with proper flags/environment variables
User->>User: Successfully install TransformerEngine
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file reviewed, 2 comments
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> Signed-off-by: jberchtold-nvidia <158520091+jberchtold-nvidia@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> Signed-off-by: jberchtold-nvidia <158520091+jberchtold-nvidia@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file reviewed, 2 comments
|
|
||
| .. code-block:: bash | ||
| export CUDNN_PATH=$(pwd)/.venv/lib/python3.12/site-packages/nvidia/cudnn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: hardcoded Python version may not work for all users - consider using a generic placeholder like pythonX.Y or explaining users should adjust this
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
|
|
||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
style: extra blank line - RST should have only one blank line before code blocks (see lines 305-306 for consistent formatting)
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Signed-off-by: jberchtold-nvidia <158520091+jberchtold-nvidia@users.noreply.github.com>
|
/te-ci |
Description
Expand the troubleshooting installation docs with a few recently debugged issues.

Type of change
Changes
uvvenvs and JAX-specific issue symptoms.Checklist: