Skip to content

Pin ptl to <2.4.0 to fix nemo bug#86

Merged
pstjohn merged 1 commit intov2-mainfrom
pstjohn/v2-main/pin-pytorch-lightning
Aug 8, 2024
Merged

Pin ptl to <2.4.0 to fix nemo bug#86
pstjohn merged 1 commit intov2-mainfrom
pstjohn/v2-main/pin-pytorch-lightning

Conversation

@pstjohn
Copy link
Collaborator

@pstjohn pstjohn commented Aug 8, 2024

No description provided.

@pstjohn
Copy link
Collaborator Author

pstjohn commented Aug 8, 2024

/build-ci

1 similar comment
@pstjohn
Copy link
Collaborator Author

pstjohn commented Aug 8, 2024

/build-ci

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you add a comment to drop this requirement once the NeMo update merges and we fix our issues with TOT? NVIDIA-NeMo/NeMo#10079

Copy link
Collaborator

Choose a reason for hiding this comment

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

also shouldn't this be in our core requirements rather than just dev? Seems like test etc should get this.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

sure, moved it to bionemo-core/requirements.txt

@jstjohn jstjohn mentioned this pull request Aug 8, 2024
Signed-off-by: Peter St. John <pstjohn@nvidia.com>
@pstjohn pstjohn force-pushed the pstjohn/v2-main/pin-pytorch-lightning branch from 5f8d748 to 86e6546 Compare August 8, 2024 22:35
@pstjohn
Copy link
Collaborator Author

pstjohn commented Aug 8, 2024

/build-ci

@pstjohn pstjohn enabled auto-merge (squash) August 8, 2024 22:36
@pstjohn
Copy link
Collaborator Author

pstjohn commented Aug 8, 2024

/build-ci

@pstjohn pstjohn merged commit c0bd2a2 into v2-main Aug 8, 2024
@pstjohn pstjohn deleted the pstjohn/v2-main/pin-pytorch-lightning branch September 16, 2024 12:54
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