Skip to content

style: fix pyink formatting on main (2 files) - #503

Open
Perseus14 wants to merge 1 commit into
mainfrom
fix/pyink-main-2
Open

Perseus14 wants to merge 1 commit into
mainfrom
fix/pyink-main-2

Conversation

@Perseus14

Copy link
Copy Markdown
Collaborator

Main currently fails the CI pyink check (.github/workflows/CPUTests.yml) on 2 files introduced by recent commits (#487 / 601381be):

  • src/maxdiffusion/configuration_utils.py
  • src/maxdiffusion/trainers/wan_trainer_2_2.py

This makes CPU CI red on PRs rebased onto main. This PR runs pyink --pyink-indentation=2 --line-length=125 on those 2 files (formatting only, no functional changes).

Verified: pyink src/maxdiffusion end_to_end --check --diff --pyink-indentation=2 --line-length=125 -> 352 files would be left unchanged.

@Perseus14
Perseus14 requested a review from entrpn as a code owner October 13, 2026 08:04
@github-actions

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces minor formatting improvements, including consolidating a multi-line list comprehension into a single line in configuration_utils.py and adding an empty line before a nested function definition in wan_trainer_2_2.py for better readability. There are no review comments, and I have no feedback to provide.

@Perseus14 Perseus14 self-assigned this Oct 13, 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.

1 participant