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

paper results directory #50

Merged
merged 1 commit into from
Jan 28, 2025
Merged

Conversation

jmframe
Copy link
Contributor

@jmframe jmframe commented Jan 15, 2025

I am adding a directory with the initial results from the HUC 01 two month simulation period. These results have been included in a dissertation, several conference presentations and in an upcoming JAWRA paper. It would be nice to have the notebook to generate the paper figure public.

Additions

  • Directory: "paper_results_jawra", which has a notebook, a figure and the HUC 01 two month LSTM animation.

Removals

  • Nothing

Changes

  • Just the paper results

Testing

  1. None required

Screenshots

Not necessary

Notes

Todos

  • None

Checklist

  • [ X] PR has an informative and human-readable title
  • [ X] Changes are limited to a single goal (no scope creep)
  • [ X] Code can be automatically merged (no conflicts)

@jmframe
Copy link
Contributor Author

jmframe commented Jan 15, 2025

@hellkite500 thanks in advance.

Copy link
Member

@aaraney aaraney left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @jmframe!

@aaraney aaraney merged commit 2316862 into NOAA-OWP:master Jan 28, 2025
6 of 7 checks passed
jmframe added a commit to jmframe/lstm that referenced this pull request Jan 29, 2025
Merge pull request NOAA-OWP#50 from jmframe/jawra_paper_results
aaraney added a commit to aaraney/lstm that referenced this pull request Feb 10, 2025
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.

2 participants