chore: bump TQ to v0.19 - #3423
Open
macandro96 wants to merge 5 commits into
Open
Conversation
macandro96
added
the
CI:Lfast
Contributor
Author
|
/ok to test 97f1b6e |
macandro96
force-pushed
the
amahishi/tq-v0.1.9-bump
branch
from
July 31, 2026 18:59
70fcd8d to
827ca05
Compare
Signed-off-by: Anish Mahishi <amahishi@nvidia.com>
Signed-off-by: Anish Mahishi <amahishi@nvidia.com>
Track Mooncake 1D promotion in durable TQ metadata so genuine singleton token columns retain their rank. Remove the obsolete pre-0.1.9 storage import fallback and add focused regression coverage. Signed-off-by: Anish Mahishi <amahishi@nvidia.com>
Densify uniform nested reads for the simple backend without applying Mooncake's singleton squeeze. Make wire-field typing explicit, test invalid shape provenance, and warn when either TQ actor runtime patch is unavailable. Signed-off-by: Anish Mahishi <amahishi@nvidia.com>
TransferQueue v0.1.9 dropped its numpy<2.0.0 pin, so the override's original justification no longer applies. The constraint it actually bypasses now is tensorrt-llm's numpy>=2.0.0,<2.4. Signed-off-by: Anish Mahishi <amahishi@nvidia.com>
macandro96
force-pushed
the
amahishi/tq-v0.1.9-bump
branch
from
July 31, 2026 20:18
827ca05 to
3f1e250
Compare
macandro96
added
CI:L1
Contributor
Author
|
/ok to test 3f1e250 |
michal2409
added a commit
to michal2409/RL
that referenced
this pull request
Aug 2, 2026
max(train/critic/loss) lands at 6.68-7.00 in CI for unrelated PRs (NVIDIA-NeMo#3401, NVIDIA-NeMo#3404, NVIDIA-NeMo#3423) since the vLLM 0.25.1 bump; the same critic-side drift is already tracked in NVIDIA-NeMo#3412. Placeholder bump, like the grad_norm bound raised in NVIDIA-NeMo#3280. Signed-off-by: Michal Futrega <mfutrega@nvidia.com>
michal2409
added a commit
to michal2409/RL
that referenced
this pull request
Aug 2, 2026
max(train/critic/loss) lands at 6.68-7.00 in CI for unrelated PRs (NVIDIA-NeMo#3401, NVIDIA-NeMo#3404, NVIDIA-NeMo#3423) since the vLLM 0.25.1 bump; the same critic-side drift is already tracked in NVIDIA-NeMo#3412. Placeholder bump, like the grad_norm bound raised in NVIDIA-NeMo#3280. Signed-off-by: Michal Futrega <mfutrega@nvidia.com>
6 tasks
9 tasks
Contributor
|
Thank you macandro96 could you pick up 2 runs for verification with tq bump up? |
Contributor
Author
@ZhiyuLi-Nvidia Yep - i believe its part of the CI which seems to have passed? Let me know if you were referring to a different run or want me to do something differently |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do ?
Issues
List issues that this PR closes (syntax):
Usage
# Add a code snippet demonstrating how to use thisBefore your PR is "Ready for review"
Pre checks:
Additional Information