Skip to content

[JAX] Add NLTK pin <3.10.1 in encoder requirements - #3314

Merged
ptrendx merged 1 commit into
NVIDIA:mainfrom
aybchan:pin-nltk-2
Aug 4, 2026
Merged

[JAX] Add NLTK pin <3.10.1 in encoder requirements#3314
ptrendx merged 1 commit into
NVIDIA:mainfrom
aybchan:pin-nltk-2

Conversation

@aybchan

@aybchan aybchan commented Aug 4, 2026

Copy link
Copy Markdown
Member

Follow up to #3306, nvbug 6553963

@greptile-apps

greptile-apps Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR constrains the JAX encoder example to NLTK versions below 3.10.1.

  • Changes the supported NLTK range from >=3.8.2 to >=3.8.2,<3.10.1.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
examples/jax/encoder/requirements.txt Adds the intended NLTK upper bound without an eligible follow-up defect.

Reviews (2): Last reviewed commit: "Add NLTK pin <3.10.1 in encoder requirem..." | Re-trigger Greptile

Signed-off-by: Alex Y. Chan <alechan@nvidia.com>
@ptrendx
ptrendx merged commit 82a0f6e into NVIDIA:main Aug 4, 2026
10 of 14 checks passed
KshitijLakhani pushed a commit that referenced this pull request Aug 4, 2026
Add NLTK pin <3.10.1 in encoder requirements

Signed-off-by: Alex Y. Chan <alechan@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants