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 ?

Fix inference configuration missing in model init.

Collection: [speechlm]

Additional Information

Signed-off-by: stevehuang52 <[email protected]>
@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

Signed-off-by: stevehuang52 <[email protected]>
tango4j
tango4j previously approved these changes Oct 30, 2025
Copy link
Collaborator

@tango4j tango4j left a comment

Choose a reason for hiding this comment

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

LGTM.

@github-actions github-actions bot removed the Run CICD label Oct 31, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2025

[🤖]: 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

@github-actions github-actions bot removed the Run CICD label Nov 1, 2025
@stevehuang52
Copy link
Collaborator Author

The fix is confirmed by user

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

[🤖]: 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

@stevehuang52 stevehuang52 merged commit 66ed216 into main Nov 3, 2025
134 checks passed
@stevehuang52 stevehuang52 deleted the heh/speechlm_infer_cfg branch November 3, 2025 17:15
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.

4 participants