-
-
Notifications
You must be signed in to change notification settings - Fork 104
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
fix: fix several bugs related to remake and symbolic indexing #583
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #583 +/- ##
==========================================
- Coverage 39.44% 0.00% -39.45%
==========================================
Files 53 53
Lines 4039 3964 -75
==========================================
- Hits 1593 0 -1593
- Misses 2446 3964 +1518 ☔ View full report in Codecov by Sentry. |
@@ -0,0 +1,118 @@ | |||
using ModelingToolkit, SymbolicIndexingInterface |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add JumpSystems to JumpProblems into this test to? (Which requires JumpProcesses?)
Note that JumpProblem is defined in JumpProcesses and has some remake related dispatches there that might need to be updated (basically they just forward all updates to the wrapped prob, i.e. jump_problem.prob
which is usually a discrete or ODE problem).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved in d4d5466
d4d5466
to
c778707
Compare
Add some inference tests as well. |
refactor: add JumpProblem to remake tests
c778707
to
7ca2602
Compare
What's next here? |
Nothing, it just needs this to be tagged first, which I assume will happen with MTKv9 |
Looks like there's a circle where tests can't pass? |
MTK tests should pass regardless of this PR |
What's the status of this one? |
This PR needs SciML/ModelingToolkit.jl#2409 to be tagged |
TODO: Symbolic remake won't work with |
CI won't pass here until SciML/ModelingToolkit.jl#2409 is merged and tagged. I also have plans to remove |
Is there any update on this or the PRs this requires before it can be merged? |
This PR is too old to be merged, and the required changes are now different than what this PR intended. I'll open a new one soon |
Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
Additional context
Add any other context about the problem here.