Skip to content

[serge] Fix 12 integration tests for model edgetam failing with import_or_config (other (12))#46907

Open
sergereview[bot] wants to merge 3 commits into
mainfrom
serge/fix/itf-23841b7d7082-158ad8ba
Open

[serge] Fix 12 integration tests for model edgetam failing with import_or_config (other (12))#46907
sergereview[bot] wants to merge 3 commits into
mainfrom
serge/fix/itf-23841b7d7082-158ad8ba

Conversation

@sergereview

@sergereview sergereview Bot commented Jun 26, 2026

Copy link
Copy Markdown

Original CI failure

  • Failure group: 1/1: 12 integration tests for model edgetamfailing withimport_or_config (other (12))
  • tests/models/edgetam/test_modeling_edgetam.py::EdgeTamModelIntegrationTest::test_inference_batched_images_batched_boxes [multi-gpu] (import_or_config, seen 5/7)

Relates to #46906

The EdgeTam integration tests fail with TypeError: FeatureListNet.forward() got an unexpected keyword argument 'original_sizes'.

Root cause: the SAM2 processor returns original_sizes for post-processing, and EdgeTamVisionModel.forward forwards all **kwargs to the TimmWrapper backbone. The underlying timm FeatureListNet does not accept original_sizes.

Fix: pop original_sizes from kwargs before calling the backbone in modular_edgetam.py, so the generated modeling_edgetam.py no longer passes it to the timm model.


This change was produced automatically by serge from a CI failure report. The patch was generated by an LLM and applied by serge; review before merging.

serge v0.1.0 · model: moonshotai/Kimi-K2.7-Code · 18 LLM turns · 30 tool calls · 74.7s · 300369 in / 9936 out tokens

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@github-actions

Copy link
Copy Markdown
Contributor

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

run-slow: edgetam

@github-actions

Copy link
Copy Markdown
Contributor

CI Dashboard: View test results in Grafana

⚠️ Code quality check failed — all test jobs were skipped. Fix the code quality issues and push again to run tests.

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