-
Notifications
You must be signed in to change notification settings - Fork 114
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate to Expronicon.jl, add Const(::Any)
#617
base: master
Are you sure you want to change the base?
Migrate to Expronicon.jl, add Const(::Any)
#617
Conversation
cc67894
to
f97813a
Compare
[skip ci]
…proniconjl-v08-from-unityperjl [skip ci]
[skip ci]
576cb9f
to
ad7a861
Compare
[skip ci]
…proniconjl-v08-from-unityperjl
a1d27e1
to
bada063
Compare
d04b925
to
53ca3b8
Compare
…proniconjl-v08-from-unityperjl
…proniconjl-v08-from-unityperjl
6bec5c2
to
73805e2
Compare
I'm posting an update on this PR. While the migration to Expronicon offers potential benefits like recursive algebraic data types and wrapping numerical values in BasicSymbolic, the work involved has proven more engineering intensive than initially anticipated. It has resulted in numerous broken tests, both within SymbolicUtils and in downstream packages, and debugging these requires a significant time investment per test. At this point, I don't have a clear timeline for completing this migration, and the performance implications are still uncertain. Due to other priorities, I'm currently putting this work on hold. I'm happy to hand this off to anyone interested in pursuing it. The PR contains the current state of the work, and I'm available to answer questions and provide context. Please comment here if you'd like to take over or discuss further. Any help would be greatly appreciated! |
No description provided.