-
-
Notifications
You must be signed in to change notification settings - Fork 234
Improved FunctionalAffect alternative #2922
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
Merged
ChrisRackauckas
merged 51 commits into
SciML:master
from
BenChung:careful-event-affects
Dec 11, 2024
Merged
Changes from 37 commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
a6f781d
First pass at MutatingFunctionalAffect
BenChung f151e42
Clarify documentation for SCC
BenChung 49d48b8
MutatingFunctionalAffect test cases
BenChung b96cd2e
More sanity checking
BenChung eec24cf
Document MutatingFunctionalAffect
BenChung fd0125d
Flip modified and observed order; write docstring
BenChung 9948de0
Run formatter
BenChung 61cf676
FIx SCC reconstruction
BenChung 8edef14
Implement initialize and finalize affects for symbolic callbacks
BenChung 3c7afd8
Test some simple initialization affects
BenChung 95956f3
Properly pass skip_checks through
BenChung 4f928ae
Support time-indexed parameters
BenChung 3eca9b9
Fix bugs relating to array arguments to callbacks
BenChung c41e7d4
Remove debug logging
BenChung 95fa1ee
Fix the namespace operation used while namespacing MutatingFunctional…
BenChung f57215a
Add support for the initializealg argument in SciMLBase callbacks
BenChung d79d49d
Switch MutatingFunctionalAffect from using ComponentArrays to using N…
BenChung c940e5e
Fix support for array forms in the NamedTuple version of MutatingFunc…
BenChung 2206425
Remove ComponentArrays dep, cleanup handling of skip_checks
BenChung 98dcd4e
Improve detection of writeback values
BenChung 3fd4462
Remove ComponentArrays dep
BenChung e6ce6ab
Rename MutatingFunctionalAffect to ImperativeAffect
BenChung 54bb95a
Fix tests
BenChung 0654909
Document ImperativeEffect and the SymbolicContinousCallback changes
BenChung aecd59b
Formatter
BenChung 89954e4
Formatter (2)
BenChung 711fb8c
Spelling
BenChung a8ea369
Spelling (2)
BenChung 6a304e7
Remove debug RGF shim
BenChung 35ec1c5
Formatter (3)
BenChung 4c3d6fc
Update docs/src/basics/Events.md
ChrisRackauckas cdddeb9
Update docs/src/basics/Events.md
ChrisRackauckas 2ac2d23
Update docs/src/basics/Events.md
ChrisRackauckas 9bf734d
Simplify callback interface, fix references
BenChung 1a3f7d4
Make array_type an actual type, though only in a limited sense
BenChung 6c7e4b8
Clean up language indocs
BenChung 06ecd2e
Format
BenChung 7f7f65c
Clear up some of the documentation language
BenChung 088f652
Merge branch 'master' into careful-event-affects
BenChung e750219
Format
BenChung a05e746
Merge branch 'master' into careful-event-affects
BenChung 3bc5d23
Fix merge issues
BenChung e69ef19
Clean up usage of custom init
BenChung 3c6b37e
Simplify setup function construction
BenChung 5fcf864
Merge branch 'master' into careful-event-affects
BenChung 16d3f5c
Refactor ImperativeAffect into its own file
BenChung 69872bf
Merge branch 'master' into careful-event-affects
BenChung 1c78dee
Fix tests & update API usage
BenChung aa556d6
Adjust quadrature test forward a little to avoid numerical issues
BenChung c169b9e
Formatter
BenChung b24e525
Update src/systems/callbacks.jl
BenChung File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
IIRC in modelica, variables you change are automatically picked as state variables, this is implied by the
reinitoperator which is mandatory for changing continuous variables in a callback. When you change an observed variable and "project" the state onto the the observed equation, there are potentially many ways you could do that? I'm trying to figure out if supporting this is something that we could come to regret in the future :)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.
Indeed it's not always possible to invert such functions.