Skip to content
This repository was archived by the owner on Nov 19, 2025. It is now read-only.

fix: dev segment in version specifier joined properly#482

Merged
terrykong merged 1 commit intomainfrom
tk/fix-package-info-version-dev-segment
Jan 16, 2025
Merged

fix: dev segment in version specifier joined properly#482
terrykong merged 1 commit intomainfrom
tk/fix-package-info-version-dev-segment

Conversation

@terrykong
Copy link
Copy Markdown
Collaborator

@terrykong terrykong commented Jan 15, 2025

What does this PR do ?

Instead of 0.6.0rc0.dev we were creating this 0.6.0rc0.d.e.v.0

Changelog

  • Please update the CHANGELOG.md under next version with high level changes in this PR.

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this 

Before your PR is "Ready for review"

Pre checks:

Checklist when contributing a new algorithm

  • Does the trainer resume and restore model state all states?
  • Does the trainer support all parallelism techniques(PP, TP, DP)?
  • Does the trainer support max_steps=-1 and validation?
  • Does the trainer only call APIs defined in alignable_interface.py?
  • Does the trainer have proper logging?

Additional Information

  • Related to # (issue)

Signed-off-by: Terry Kong <terryk@nvidia.com>
@terrykong terrykong requested review from ashors1 and ko3n1g January 15, 2025 20:19
@terrykong terrykong added the Run CICD Set + un-set to retrigger (add after r*.*.* labels) label Jan 15, 2025
@terrykong terrykong merged commit 735c415 into main Jan 16, 2025
@terrykong terrykong deleted the tk/fix-package-info-version-dev-segment branch January 16, 2025 21:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Run CICD Set + un-set to retrigger (add after r*.*.* labels)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants