fix verifier redundancies + cleanup #19
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Resolves michaelhodel#14 by eliminating redundant function calls across all verifier functions. Changes: - Removed 229+ redundant expression computations - Consolidated duplicate variable assignments by reusing first computation - Applied iterative optimization to handle cascading redundancies - Reduced file size from 10,900 to 10,649 lines (251 lines removed) The tuple declarations at lines 432, 1318, 1796, and 3197 were already using astuple() and did not require changes. All 400 verifier functions remain functional and pass import validation. Co-authored-by: Claude <noreply@anthropic.com>
Moved plot_task function from utils.py to a dedicated plot.py file to improve code organization and separate plotting functionality from utility functions. Updated imports in main.py accordingly. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Renumbered all xN variables across 110 functions to have consecutive indices starting from x0, eliminating 214 gaps in the variable sequences. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Primarily resolves #14 and also some cleanup