Skip to content

Conversation

@ple13
Copy link
Contributor

@ple13 ple13 commented Dec 12, 2025

This uses a mock for the Solver to avoid having to maintain inputs that get the exact desired response from the real Solver. That would otherwise be difficult to keep up-to-date with changes to the underlying solver implementations.

@wfa-reviewable
Copy link

This change is Reviewable

@ple13 ple13 requested a review from SanjayVas December 12, 2025 20:43
Copy link
Member

@SanjayVas SanjayVas left a comment

Choose a reason for hiding this comment

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

@SanjayVas reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ple13)

Copy link
Collaborator

@stevenwarejones stevenwarejones left a comment

Choose a reason for hiding this comment

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

@stevenwarejones reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @ple13)

@SanjayVas SanjayVas force-pushed the lephi-fix-large-correction-test branch from ac164c3 to 7e0952e Compare December 12, 2025 22:13
@SanjayVas SanjayVas enabled auto-merge (squash) December 12, 2025 22:13
@SanjayVas SanjayVas merged commit 3da6cf2 into main Dec 12, 2025
7 checks passed
@SanjayVas SanjayVas deleted the lephi-fix-large-correction-test branch December 12, 2025 22:26
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.

5 participants