-
-
Notifications
You must be signed in to change notification settings - Fork 210
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
Refactor docs with @mtkmodel
#2286
Conversation
What's left to merge? |
Could you retrigger CI @ChrisRackauckas? |
Needs to be resolved to trigger. |
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.
Fix for the SI tutorial.
k5; k6; k7; k8; k9; k10 | ||
end | ||
begin | ||
D = Differential(t) |
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.
D = Differential(t) | |
@variables t | |
D = Differential(t) |
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.
Will resolve the conflicts and commit this in.
@@ -66,8 +72,7 @@ Let's try to check specific parameters and their combinations | |||
|
|||
```@example SI | |||
to_check = [k5, k7, k10 / k9, k5 + k6] |
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.
to_check = [k5, k7, k10 / k9, k5 + k6] | |
@unpack k5, k6, k7, k9, k10 = de | |
to_check = [k5, k7, k10 / k9, k5 + k6] |
I'm refactoring the "Getting started" based on Friday's discussion. I'm planning to make a copy of it with |
I think the Having docstrigns for both would also be a plus. |
e76e7dc
to
5ccf94c
Compare
5ccf94c
to
fa153f0
Compare
Note that these have changed since the previous push:
|
What does it take to fix the Documentation CI? |
what's left to merge? |
- All sections, except building hierarchal models, use `@mtkmodel`. - "Non-DSL way of defining an ODESystem" section is added.
…nts with `@mtkmodel` and connectors with `@connectors`" to indicate that it describes `@connector`s too.
I'll retrigger docs build. |
fa153f0
to
94a4f06
Compare
- while here, update the description on setting unit metadata in Contextual-Variables
94a4f06
to
2ef4752
Compare
Let's merge and follow up with more changes. |
@mtkmodel
and adds a note on usingcomplete
Model.structure
is.@mtkmodel
connect
section in Variable metadataget_unit
andVariableUnit