Skip to content

[CI] Clean up test_modeling_common.py - #14988

Merged
DN6 merged 2 commits into
mainfrom
remove-unet-mixin
Oct 14, 2026
Merged

DN6 merged 2 commits into
mainfrom
remove-unet-mixin

Conversation

@DN6

@DN6 DN6 commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

  • Remove old UNetTesterMixin
  • Fix a broken import path in test generating script.

Fixes # (issue)

Before submitting

  • Did you use an AI agent (Claude Code, Codex, Cursor, etc.) to help with this PR? If so:
    • Did you read the Coding with AI agents guide?
    • Did you run the self-review skill on the diff?
    • Did you share the final self-review notes in the PR description or a comment?
  • Did you read the contributor guideline?
  • Did you read our philosophy doc? (important for complex PRs)
  • 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?
  • Are you the author (or part of the team) of the model/pipeline (only applicable for model/pipeline related PRs)?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

@DN6
DN6 requested a review from sayakpaul October 8, 2026 12:00
@github-actions github-actions Bot added tests size/M PR with diff < 200 LOC labels Oct 8, 2026

@sayakpaul sayakpaul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@DN6
DN6 merged commit ea981b8 into main Oct 14, 2026
47 of 49 checks passed
@DN6
DN6 deleted the remove-unet-mixin branch October 14, 2026 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M PR with diff < 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants