Skip to content

Conversation

@sarahyurick
Copy link
Contributor

No description provided.

Signed-off-by: Sarah Yurick <sarahyurick@gmail.com>
@sarahyurick sarahyurick added the gpuci Run GPU CI/CD on PR label Feb 7, 2025
pyproject.toml Outdated
"spacy>=3.6.0, <3.8.0",
# TODO: Remove this pin once newer version is released
"transformers==4.46.3",
"transformers",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we pin to say not include broken version ?

Suggested change
"transformers",
"transformers!=4.47.0",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, although I think it was fixed by 4.48.0 so I might try transformers>=4.48.0.

Copy link
Contributor

Choose a reason for hiding this comment

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

I will also check if the upper bound works with Nvidia's NeMo container. If it works, then a upper pin is great, If it does not a more permissive pin (which just prevents 1 version) might be the way to go.

sarahyurick and others added 2 commits February 10, 2025 12:08
Signed-off-by: Sarah Yurick <sarahyurick@gmail.com>
@sarahyurick sarahyurick added gpuci Run GPU CI/CD on PR and removed gpuci Run GPU CI/CD on PR labels Feb 10, 2025
@ryantwolf ryantwolf removed the gpuci Run GPU CI/CD on PR label Feb 11, 2025
@sarahyurick sarahyurick changed the title Unpin Transformers version Pin Transformers version >= 4.48.0 Feb 11, 2025
@sarahyurick sarahyurick merged commit 334a331 into NVIDIA-NeMo:main Feb 11, 2025
5 of 6 checks passed
@sarahyurick sarahyurick deleted the unpin_transformers branch February 28, 2025 18:31
jnke2016 pushed a commit to jnke2016/Curator that referenced this pull request Nov 12, 2025
* Unpin Transformers version

Signed-off-by: Sarah Yurick <sarahyurick@gmail.com>

* add lower bound

Signed-off-by: Sarah Yurick <sarahyurick@gmail.com>

---------

Signed-off-by: Sarah Yurick <sarahyurick@gmail.com>
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