Skip to content

CSE improvement#134

Merged
tBuLi merged 13 commits into
masterfrom
cse_fix
Mar 25, 2026
Merged

CSE improvement#134
tBuLi merged 13 commits into
masterfrom
cse_fix

Conversation

@tBuLi

@tBuLi tBuLi commented Mar 25, 2026

Copy link
Copy Markdown
Owner

This PR changes the current sympy based CSE to a direct port of GAmphetamine's CSE. This port is well worth it because this is the only CSE known to produce the hand optimized solutions in known cases in 3DPGA.

Todo:

  • Port polynomial.js CSE code
  • Update docs
  • Maintain the sympy based code printing path as we still need it in Torch compatibility #117 and other places where we want to print non-python code.
  • Use the new CSE in the SymPy path.
  • Connect the new CSE properly to the existing printing infrastructure.

@tBuLi tBuLi marked this pull request as ready for review March 25, 2026 11:32
@tBuLi tBuLi merged commit e8406cf into master Mar 25, 2026
11 of 12 checks passed
@tBuLi tBuLi deleted the cse_fix branch March 25, 2026 19:02
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