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
Full nosnoc pipeline for pss using the Stewart reformulation #90
base: main
Are you sure you want to change the base?
Conversation
9db9e26
to
a8b70dc
Compare
91b252a
to
8621312
Compare
8621312
to
88229d4
Compare
The new structure is very clean and well-organized. Before we merge, I have a few comments to polish it further:
|
A few responses here:
My opinion is this is fine. I prefer the clean-ness of
I think this is something to be tackled in the near future (pre v1.0?). I will write a quick script to automatically update the BSD licence header in the required
Agreed
What do you mean by "removed". There are two ways to do that for objects of classes: Actually removing via DynamicProps or just modifying the display to "hide" the unused properties via custom display functions. I use both to accomplish the results you see when you display
I really prefer opts to options, but I will concede the point here.
Because we use
No disagreement here.
On purely aesthetics I agree, but at the same time this is useful to separate out classes from functions: Classes are capitalized where as function are not.
The idea here was to separate model definition from OCP definition, i,e. model only contains things necessary for dynamical definition of the system and an ocp class contains the necessary things for an OCP. I believe this idea was discussed and rejected previously.
Yes currently it is passed that way. I have no strong opinions on storing it as
|
No description provided.