Skip to content

Conversation

@stevehuang52
Copy link
Collaborator

Important

The Update branch button must only be pressed in very rare occassions.
An outdated branch is never blocking the merge of a PR.
Please reach out to the automation team before pressing that button.

What does this PR do ?

Replace pcikle.loads() with json.loads() to avoid potential security issues

Collection: [speechlm, multimodal]

PR Type:

  • New Feature
  • Bugfix
  • Documentation

If you haven't finished some of the above items you can still open "Draft" PR.

Signed-off-by: stevehuang52 <heh@nvidia.com>
@github-actions
Copy link
Contributor

[🤖]: Hi @stevehuang52 👋,

We wanted to let you know that a CICD pipeline for this PR just finished successfully.

So it might be time to merge this PR or get some approvals.

//cc @chtruong814 @ko3n1g @pablo-garay @thomasdhc

@nithinraok nithinraok merged commit aecb5d3 into main Dec 31, 2025
256 checks passed
@nithinraok nithinraok deleted the heh/speechlm_cleanup branch December 31, 2025 16:04
chtruong814 pushed a commit that referenced this pull request Jan 5, 2026
Signed-off-by: stevehuang52 <heh@nvidia.com>
Signed-off-by: Charlie Truong <chtruong@nvidia.com>
chtruong814 pushed a commit that referenced this pull request Jan 6, 2026
Signed-off-by: stevehuang52 <heh@nvidia.com>
Signed-off-by: Charlie Truong <chtruong@nvidia.com>
chtruong814 added a commit that referenced this pull request Jan 6, 2026
Signed-off-by: stevehuang52 <heh@nvidia.com>
Signed-off-by: Charlie Truong <chtruong@nvidia.com>
Co-authored-by: He Huang (Steve) <105218074+stevehuang52@users.noreply.github.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.

3 participants