-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add function to return vector_field of continuous system #201
Conversation
…ematicalSystems.jl into ueliwechsler/160
Is this still WIP? |
Sorry, I did not have that much time to work on this PR in the last weeks. The only thing that is left to do is the following:
I am not really used to the wording of continuous systems and, to be honest, wording in general 😅. If you could provide some help, that would be really great. But apart from adding documentation to the MathematicalSystems.jl/src/instantiate.jl Lines 215 to 237 in 74745b4
VectorField this PR is fine. And should not be WIP.
|
Alright, thanks for the write-up. I'm enthusiastic about this PR as it gives access to |
Finally, I finished my work on |
Can you split up the file |
Sure. I can create a file |
Sounds good. I do not have suggestions for a folder name, but I also think we do not need a folder. |
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.
lgtm, thanks!
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.
Thanks!
The VectorField
struct is not tested I think.
I added the tests. Should be fine now. |
I haven't done this in a while. Can I just merge or do I have to rebase or similar? |
Let's wait if @blegat has something to say. If you have the rights to do so, you can just merge (no conflict here). |
Merging! |
Closes #160.
vector_field
to systems of typeAbstractContinuousSystem
.successor
method to make it more generic (no need to add a new method for every new type)mapping
for retrieving the parametric function:p
or blackbox function:f
. (function name is WIP though)isblackbox
traitVectorField
type