Skip to content

Fix skipped test_model_xattn_padding test by updating mask padding logic - #14710

Open
xiaoyaoqilan wants to merge 1 commit into
huggingface:mainfrom
xiaoyaoqilan:doc-fix-14699
Open

Fix skipped test_model_xattn_padding test by updating mask padding logic#14710
xiaoyaoqilan wants to merge 1 commit into
huggingface:mainfrom
xiaoyaoqilan:doc-fix-14699

Conversation

@xiaoyaoqilan

Copy link
Copy Markdown

What does this PR do?

This PR fixes the documentation issue reported in #14699: corrects remaining_length to target_length in tests/models/unets/test_models_unet_2d_condition.py.

Fixes #14699

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.

Fixes #14699

@github-actions github-actions Bot added fixes-issue tests size/S PR with diff < 50 LOC and removed fixes-issue labels Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S PR with diff < 50 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix skipped test_model_xattn_padding test by updating mask padding logic

1 participant