Skip to content

[fix][ci] Format Tinker engine test - #2002

Open
bvolpato wants to merge 1 commit into
NovaSky-AI:mainfrom
bvolpato:bvolpato/format-tinker-test
Open

[fix][ci] Format Tinker engine test#2002
bvolpato wants to merge 1 commit into
NovaSky-AI:mainfrom
bvolpato:bvolpato/format-tinker-test

Conversation

@bvolpato

@bvolpato bvolpato commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the missing top-level function separator in tests/tinker/test_engine.py so Black leaves the repository unchanged.

This fixes the code-quality failure affecting pull requests based on current main.

Testing

  • pre-commit run --all-files
  • uv run --isolated --extra dev --extra tinker --extra jax pytest tests/tinker/test_engine.py -q
    • 16 passed

Unblocks #1998.

@bvolpato
bvolpato marked this pull request as ready for review August 7, 2026 01:14

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

Copy link
Copy Markdown
Contributor

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 adjustments in tests/tinker/test_engine.py by adding blank lines before the forward_backward_payload function. There are no review comments, and I have no feedback to provide.

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