Skip to content

Fix failing MusicgenStereo integration tests#44527

Open
Sai-Suraj-27 wants to merge 5 commits intohuggingface:mainfrom
Sai-Suraj-27:fix_failing_perceptronlm_test
Open

Fix failing MusicgenStereo integration tests#44527
Sai-Suraj-27 wants to merge 5 commits intohuggingface:mainfrom
Sai-Suraj-27:fix_failing_perceptronlm_test

Conversation

@Sai-Suraj-27
Copy link
Contributor

@Sai-Suraj-27 Sai-Suraj-27 commented Mar 8, 2026

What does this PR do?

Fixes these failing MusicgenStereoIntegrationTests

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you read the contributor guideline,
    Pull Request section?
  • Was this discussed/approved via a Github issue or the forum? Please add a link
    to it if that's the case.
  • Did you make sure to update the documentation with your changes? Here are the
    documentation guidelines, and
    here are tips on formatting docstrings.
  • Did you write any new necessary tests?

Who can review?

@Rocketknight1 @vasqu

@Sai-Suraj-27 Sai-Suraj-27 changed the title Fix failing PerceptionLMForConditionalGenerationIntegrationTests Fix failing PerceptionLM integration tests Mar 8, 2026
@Sai-Suraj-27 Sai-Suraj-27 changed the title Fix failing PerceptionLM integration tests Fix failing PerceptionLM & MusicgenStereo integration tests Mar 9, 2026
Copy link
Contributor

@vasqu vasqu left a comment

Choose a reason for hiding this comment

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

Auto dtype strikes again 👀

@vasqu
Copy link
Contributor

vasqu commented Mar 9, 2026

run-slow: musicgen, perception_lm

@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

Workflow Run ⚙️

This comment contains run-slow, running the specified jobs:

models: ["models/musicgen", "models/perception_lm"]
quantizations: []

@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

CI Results

Workflow Run ⚙️

Commit Info

Context Commit Description
RUN 6a370326 workflow commit (merge commit)
PR 370441e5 branch commit (from PR)
main 70162852 base commit (on main)

Model CI Report

2 new failed tests from this PR 😭

  • perception_lm:
    tests/models/perception_lm/test_modeling_perception_lm.py::PerceptionLMForConditionalGenerationIntegrationTest::test_small_model_integration_test (❌ ⟹ ❌)
    tests/models/perception_lm/test_modeling_perception_lm.py::PerceptionLMForConditionalGenerationIntegrationTest::test_small_model_integration_test_batched (❌ ⟹ ❌)

@Sai-Suraj-27
Copy link
Contributor Author

Okay, The dtype mismatch issue is fixed but PerceptionLM is generating pretty weird output. Sorry couldn't test this before because this model requires some special access which I don't have.

@vasqu
Copy link
Contributor

vasqu commented Mar 9, 2026

Let's revert the perceptron tests then and go with the musicgen fixes then, no worries. Looks like perceptron will need a deep investigation 👀

@Sai-Suraj-27
Copy link
Contributor Author

Let's revert the perceptron tests then and go with the musicgen fixes then, no worries. Looks like perceptron will need a deep investigation 👀

Done. Will try to check the perception-lm failure later sometime.

@github-actions
Copy link
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: musicgen

@Sai-Suraj-27 Sai-Suraj-27 changed the title Fix failing PerceptionLM & MusicgenStereo integration tests Fix failing MusicgenStereo integration tests Mar 10, 2026
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.

2 participants