Skip to content

Conversation

@LoserCheems
Copy link
Collaborator

Changes the boolean flag from False to True for the largest input size test case, allowing the forward equivalence benchmark to include 512x512 resolution testing in its validation suite.

Changes the boolean flag from False to True for the largest input size test case, allowing the forward equivalence benchmark to include 512x512 resolution testing in its validation suite.
Copilot AI review requested due to automatic review settings July 1, 2025 04:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables testing for a 512x512 input resolution by updating the boolean flag in the test case configuration.

  • Changed the 512x512 test case flag from False to True
  • Expands the forward equivalence benchmark to cover higher resolution inputs
Comments suppressed due to low confidence (1)

benchmarks/benchmark_forward_equivalence.py:380

  • Verify that the system has adequate resources (e.g., GPU memory) to support testing at the 512x512 resolution, ensuring that enabling this test case does not inadvertently cause resource constraints.
        (1, 2, 1, 512, 512, 128, True),

@LoserCheems LoserCheems merged commit efa1ba6 into main Jul 1, 2025
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