Skip to content
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

API docs update, removed excess imports, removed cite_msx #136

Merged
merged 9 commits into from
Nov 10, 2023

Conversation

kaklise
Copy link

@kaklise kaklise commented Nov 10, 2023

No description provided.


* `Arsenic Chloramine model <https://github.com/dbhart/WNTR/blob/msx/wntr/msx/_library_data/arsenic_chloramine.json>`_ [SRU23]_
* `Lead Plumbosolvency model <https://github.com/dbhart/WNTR/blob/msx/wntr/msx/_library_data/lead_ppm.json>`_ [BWMS20]_
[TODO change the 2 links to usepa, add other models if they are referenced]
Copy link
Owner

@dbhart dbhart Nov 10, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kaklise FYI - I have figured out how to include extra files from the source area into the documentation build. I.e., we can reference them using relative paths in the deployed documentation without having to reference github.com/.io at all in this spot. After I pull it in, I can make that change (I'm seeing merge conflicts if I do it early)

@dbhart dbhart merged commit 9bc4140 into dbhart:msx Nov 10, 2023
0 of 2 checks passed
dbhart pushed a commit that referenced this pull request Dec 16, 2024
* Add scheduled testing to workflow

* MSX documentation update

* minor updates

* API docs update, removed excess imports, removed cite_msx

---------

Co-authored-by: Kirk Bonney <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants