-
Notifications
You must be signed in to change notification settings - Fork 5
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
Run quick start in docs #1811
Run quick start in docs #1811
Conversation
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.
Looking good
docs/reference/node/basin.qmd
Outdated
potential_evaporation | Float64 | $\text{m}/\text{s}$ | non-negative | ||
drainage | Float64 | $\text{m}^3/\text{s}$ | non-negative | ||
infiltration | Float64 | $\text{m}^3/\text{s}$ | non-negative | ||
precipitation | Float64 | $\text m}/\text{s}$ | non-negative |
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.
This removes the opening braces.
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.
Reverted the commit, and fixed in 3c86334.
I tried find and replace with regex in VS Code, but couldn't find any matches, probably since the notebooks are JSON encoded. Though if I then searched for the JSON encoding in VS Code that was also buggy, so I did it in trusty Notepad++
This reverts commit 394df61.
Fixes #1807.
Rather than having docs with code that doesn't run and output in static images, this intends to run the notebook as part of the docs generation just like example.ipynb.
I am also making more edits which I'll describe later.