Skip to content

feat: (non-container) update pyproject.toml, upgrade TE to 2.10.0#1090

Closed
hemildesai wants to merge 15 commits into
mainfrom
hemil/update-pyproject
Closed

feat: (non-container) update pyproject.toml, upgrade TE to 2.10.0#1090
hemildesai wants to merge 15 commits into
mainfrom
hemil/update-pyproject

Conversation

@hemildesai

Copy link
Copy Markdown
Contributor

Allows installing all dependencies for using uv sync

For custom MoEs, uv sync --extra moe will install everything necessary to run them.

Switch torch source to { index = "pytorch-cu130", marker = "sys_platform == 'linux'" } if your system has CUDA 13

Signed-off-by: Hemil Desai <hemild@nvidia.com>
Signed-off-by: Hemil Desai <hemild@nvidia.com>
@hemildesai hemildesai requested a review from a team as a code owner January 20, 2026 06:31
@copy-pr-bot

copy-pr-bot Bot commented Jan 20, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Signed-off-by: Hemil Desai <hemild@nvidia.com>
Signed-off-by: Hemil Desai <hemild@nvidia.com>
@hemildesai

Copy link
Copy Markdown
Contributor Author

/ok to test 3ecbf91

Signed-off-by: Hemil Desai <hemild@nvidia.com>
@hemildesai

Copy link
Copy Markdown
Contributor Author

/ok to test 8a0ddb4

Signed-off-by: Hemil Desai <hemild@nvidia.com>
@hemildesai

Copy link
Copy Markdown
Contributor Author

/ok to test 10a2116

@akoumpa

akoumpa commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

@NVIDIA-NeMo/automation PTAL thank you

Comment thread docker/Dockerfile Outdated
COPY . /opt/Automodel
RUN cd /opt/Automodel && \
if [ "$BASE_IMAGE" = "pytorch" ]; then \
sed -i '/override-dependencies = \["nvidia-cudnn-cu12/d' pyproject.toml && \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@hemildesai this override already exists, please see docker/common/uv-pytorch.toml

thomasdhc and others added 3 commits January 26, 2026 07:24
Signed-off-by: Dong Hyuk Chang <donghyukc@nvidia.com>
Signed-off-by: Dong Hyuk Chang <donghyukc@nvidia.com>
Signed-off-by: thomasdhc <thomasdhc@users.noreply.github.com>
@thomasdhc

Copy link
Copy Markdown
Contributor

/ok to test 911877e

thomasdhc
thomasdhc previously approved these changes Jan 26, 2026
Signed-off-by: Dong Hyuk Chang <donghyukc@nvidia.com>
thomasdhc and others added 2 commits January 26, 2026 08:10
Signed-off-by: Dong Hyuk Chang <donghyukc@nvidia.com>
Signed-off-by: thomasdhc <thomasdhc@users.noreply.github.com>
@thomasdhc

Copy link
Copy Markdown
Contributor

/ok to test 7f879b0

@thomasdhc

Copy link
Copy Markdown
Contributor

Closing in favor of: #1127

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