Skip to content

refactor output tracing in mobilenet_v1 - #43999

Closed
omkar-334 wants to merge 3 commits into
huggingface:mainfrom
omkar-334:refactor-mobilenetv1
Closed

omkar-334 wants to merge 3 commits into
huggingface:mainfrom
omkar-334:refactor-mobilenetv1

Conversation

@omkar-334

@omkar-334 omkar-334 commented Feb 14, 2026 •

Copy link
Copy Markdown
Contributor

This PR refactors the mobilenet_v1 model as per #43979

cc @molbap

@omkar-334

Copy link
Copy Markdown
Contributor Author

I changed the import and updated the test to fix the failing tests, but 1 test is still failing.

@github-actions

Copy link
Copy Markdown
Contributor

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

run-slow: mobilenet_v1

@ArthurZucker

Copy link
Copy Markdown
Collaborator

Thanks for picking this up from #43979! That call to contributions has been closed, and the reference refactor (#43590) landed afterwards with a slightly different pattern, so this branch is out of date and I'm closing it. If you'd still like to migrate MobileNetV1, a fresh PR following #43590 is welcome.

@omkar-334

Copy link
Copy Markdown
Contributor Author

@ArthurZucker I think this model MobileNetV1 is covered in #48378 by @sbucaille ? So I think a fresh PR is not needed? please correct me if im wrong , happy to contribute to this

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