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.

Sign up for GitHub

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

Jump to bottom

Updates to the RnR Test API #846

Closed
wants to merge 64 commits into from
Closed

Updates to the RnR Test API #846

wants to merge 64 commits into from

Conversation

taddyb
Copy link

@taddyb taddyb commented Dec 4, 2024

Additions:

  • removed the global level exposure of nwm_routing.main_v04 as it's exposed from compilation
  • fixed a test case in the API for the LowerColorado procedure

Tests:

  • verified the API tests work
  • pushed a new docker container up to packages

taddyb and others added 30 commits August 20, 2024 20:49
@taddyb
migrated fastapi wrapper to the troute branch
c22c82a
@taddyb
Merge pull request #1 from taddyb33/migration
43d4489 
migrated fastapi wrapper to the troute branch
@taddyb
ensuring this dir is here
c7cc896
@taddyb
added restart files to remove dry conditions for RFC routing
a2c826f
@taddyb
Merge branch 'development' of https://github.com/taddyb33/t-route-dev …
4eba8a9 
…into migration
@taddyb
Merge pull request #2 from taddyb33/migration
60f2c7c 
Migration
@taddyb
Added docs for t-route api release
7306c68
@taddyb
Merge branch 'development' of https://github.com/taddyb33/t-route-dev …
60986b8 
…into migration
@taddyb
linted code?
510386e
@taddyb
Merge pull request #3 from taddyb33/migration
3dfb149 
Documentation
@taddyb
added test files
0a6f481
@taddyb
Merge pull request #4 from taddyb33/test_docs
627d57f 
added test files
@taddyb
added author name in app/main.py, added specifics to testing README f…
2cf4c3f 
…or API
@taddyb
changed port from 8004 to 8000
81fe066
@taddyb
added the lower colorado forcing and domain files into the api test
d0f8575
@taddyb
Merge branch 'NOAA-OWP:master' into development
35e4b6c
@AminTorabi-NOAA @taddyb
if the folder doesn't exist
ffb9f96
@AminTorabi-NOAA @taddyb
handling edge cases
b28544f
@JurgenZach-NOAA @taddyb
Update readme.md: removed obsolete no-e option (NOAA-OWP#843)
2ecd279 
The non-editable flag:
./compiler.sh no-e
is no longer necessary 

(see NOAA-OWP#675 for details)
@AminTorabi-NOAA @taddyb
Speeding up writing output (NOAA-OWP#841)
2deff3c
@taddyb
Merge branch 'rebase' of https://github.com/taddyb33/t-route-dev into…
51b2253 
… rebase
@taddyb
removing lower colorado duplicated data
b3960e0
@taddyb
Merge pull request #5 from taddyb33/rebase
5e1a004 
Removing lower colorado duplicated files
Delete data/troute_restart/.gitkeep
884c101
@taddyb
completed general PR comments and formatting
ea96782
@taddyb
Merge branch 'development' of https://github.com/taddyb33/t-route-dev …
7e5dc45 
…into development
@taddyb
added general LowerColorado Test case for API
4fc70fa
@taddyb
reorganized dockerfiles
8ca681a
@taddyb
updated docs, added README
7c20da5
@taddyb
adding tests for water transfer
dbb06c5
taddyb added 28 commits November 15, 2024 08:54
@taddyb
added notebook dockerfile for OE testing
d2d03ec
@taddyb
found a bug in discharge calc
2243c34
@taddyb
deleting unneccessary files:
1304f11
@taddyb
adding saved change to notebook
14059a2
@taddyb
Update dockerfile_notebook.md
e96ca82
@taddyb
adding support for better last obs handling
f62980e
@taddyb
Merge branch 'old-river-lock' of https://github.com/taddyb/t-route in…
1ad5ad9 
…to old-river-lock
@taddyb
Merge branch 'development' of https://github.com/taddyb/t-route into …
599329f 
…development
@taddyb
reformatted plots
15022c1
@taddyb
squashed merge from development
9113a07
@taddyb
reformatted requirements and .gitignore
b9270d5
@taddyb
reformatting from pull
cdd3a29
@taddyb
Merge pull request #4 from taddyb/old-river-lock
55d92f1 
Old river lock
@taddyb
adding test f files
2759195
@taddyb
added pyproject.toml
2e5ec75
@taddyb
removing build dir
8cfdc97
@taddyb
fixed the troute-config unit tests
ef0f4b5
@taddyb
added unit tests for network
fe4794e
@taddyb
removed old tests and formatted unit tests for t-route routing
198d6ef
@taddyb
added docs to doc/
03cf302
@taddyb
added change to allow v2.2 to work
be96621
@taddyb
fixed the v2.2 plotting code
98af87f
@taddyb
added BMI test, missing two dataframes
2901799
@taddyb
adding bmi test
f9c53c2
@taddyb
removing utils
def0388
@taddyb
Merge pull request #5 from taddyb/fims_unit_tests
e09098c 
Unit Test Deliverables
@taddyb
added changes to t-route API to support v2.2
c708db8
@taddyb
removed __main__ call from init, and fixed the t-route tests
67e0fff
@taddyb taddyb closed this Dec 4, 2024
@taddyb taddyb deleted the rnr branch December 4, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@taddyb @AminTorabi-NOAA @JurgenZach-NOAA