Merge CUAHSI/cssi_evaluation to hydroframe/cssi_evaluation by igarousi · Pull Request #17 · hydroframe/cssi_evaluation

igarousi · 2026-03-17T12:02:59Z

This update reorganizes the repository around a clearer hydrologic model evaluation workflow. The main changes include a cleaner source layout, expanded example notebooks for both NWM and ParFlow workflows, updated environment setup instructions, and revised documentation with visuals that explain the repository structure and workflow.

Main Changes:

Reorganized the repository to better reflect the evaluation workflow
Clarified the internal package structure under src/cssi_evaluation/, including model adapters, data-access helpers, shared utilities, and variable-specific diagnostics.
Added and updated example notebooks for multiple workflows, including:
- NWM streamflow evaluation
- ParFlow streamflow evaluation
- SWE-focused examples for ParFlow
- data collection examples for observations and HydroData-based workflows
Updated the main README to better explain the purpose of the repository, the workflow, and how the major components fit together.
Added visuals to the README, making the framework easier to understand for new users.
Improved dependency and environment setup instructions, including a more consistent installation path for package, notebook, and development dependencies.

…o functions: computing water year, and performing a same-day swe comparison.

…nto develop

…ample notebook into a data collection notebook and a single site comparison notebook

…ts and code

…ded steps in getting_started.md

…for circular import in notebook 02

… for popup formatting

…ion changes for notebook 02 sections 4-5

…mmary and plots, condon diagrams

…lation/ablation compute_stats

… a watershed)

…ity to plot_sites_within_domain() function

…ems using getCCSSData() on Verde

…works on Verde up to the 'Summary Metrics at Multiple Sites' section

…n & first plotting

…rodata_utils; removed most of the Cali files

Cam 968 snow notebook 03 - Spatial Snow Analysis

…luding model specific and some observational functions.

…riginal 01_data_collection.ipynb notebooks. Renamed these for PF/HydroData and NWM

…nfirm notebook works.

…k works.

…the content is specific to Parflow. We can re-use the content of this file later when the reorganization of the github repository is finalized.

…at supports both the core library configuration for the evaluation workflow/package and the runtime dependencies.

…ks fine. NWM and Parflow notebooks (#2) should be updated. They still point to cssi_evaluation.snow which no longer exists in our reorg branch

… all dependencies is already mentioned in getting_started.md

Merge reorg branch into develop branch

Merge develop branch into main branch

Merged to be synced with the upstream.

pyproject.toml

cssi_env.yml

pyproject.toml

amy-defnet

Thanks for the changes, looks good! I'll handle the failing tests on a separate PR.

igarousi and others added 30 commits October 16, 2025 14:55

added the nwm snow comparison notebook

cf40f2d

added documentation for snow metrics

e101a78

uploadedc the correct documentation files

2dec89e

removed the wrong document

8bccd71

added snow.utils module to the evaluation framework. This contains tw…

b15138c

…o functions: computing water year, and performing a same-day swe comparison.

Merge branch 'develop' of https://github.com/CUAHSI/cssi_evaluation i…

c2799b3

…nto develop

added different day comparison function to snow utils

7a43a15

formatting

90e34d3

added snow melt functions to the snow utils module. Split the snow ex…

b761ee1

…ample notebook into a data collection notebook and a single site comparison notebook

cleaned single site snow comparison notebook by removing unused impor…

4d886d5

…ts and code

completed setting up virtual environment on Verde with micromamba, ad…

0490f8c

…ded steps in getting_started.md

created a new 01 snow notebook for HydroData data retrieval

875ca33

changes to package versions for importing packages and snow __init__ …

f66ac6a

…for circular import in notebook 02

added html/css styling in nwm_utils plot_sites_within_domain function…

14389f0

… for popup formatting

changed text box formatting, clarified language around 'model outputs'

1286f19

remove notebook output

2762686

formatting for comparison_plots() function in nwm_utils.py

256ed1b

added timeseries/scatter plot figure description text

bbc89d3

Reorganized and added sections for notebook 02; Plotting and descript…

c112d45

…ion changes for notebook 02 sections 4-5

cleaned up section 5 of 02 notebook (pre stats section 6 work)

4ea1f54

02 snow notebook outline for stats section, added multi-site stats su…

2a69c8b

…mmary and plots, condon diagrams

changed 'relative' bias and 'mean' bias descriptors, fixed the accumu…

ea9ad62

…lation/ablation compute_stats

Created a third notebook for the last part of snow comparison (across…

1aea0c7

… a watershed)

add .DS_store and .ipynb_checkpoints to gitignore

509f549

create 01_data_collection_HydroData.ipynb notebook file

a2002ce

Added getting shapefile from WBD HUC dataset; added Pandas functional…

cd8a471

…ity to plot_sites_within_domain() function

added geojson file to domain_data for CCSS stations. Still have probl…

cd3acc0

…ems using getCCSSData() on Verde

Added geojson CCSS file for use on Verde. Confirmed that notebook 02 …

cb7b3a0

…works on Verde up to the 'Summary Metrics at Multiple Sites' section

wrote loop for accessing CONUS1 SWE with the get_gridded_data functio…

0044fca

…n & first plotting

nearly final Hydrodata notebook; finalized section outline; added hyd…

075b4bd

…rodata_utils; removed most of the Cali files

igarousi and others added 23 commits March 9, 2026 08:40

Merge pull request #7 from CUAHSI/CAM-968-Snotebook03

e53cc5a

Cam 968 snow notebook 03 - Spatial Snow Analysis

moved files to new directories

c37b066

Created new folders for reorganizing files

b1d92f0

Added all necessary python files, init files; Added a General README

1bfb001

added header descriptions in empty files

1515a62

Migrated most functions to their appropriate files for reorg. Not inc…

b00bf9c

…luding model specific and some observational functions.

Checked package imports for all new files, except model specific ones

9867292

created 'collect_observations' directory in /examples and moved the o…

62b8d02

…riginal 01_data_collection.ipynb notebooks. Renamed these for PF/HydroData and NWM

edited files for missing imports requirements

b2fc772

Modify dataCollectionHydrodata_parflow.ipynb for module reorg. Can co…

72319fb

…nfirm notebook works.

Modify dataCollection_nwm.ipynb for module reorg. Can confirm noteboo…

da89611

…k works.

final changes before PR

c91ca57

Moved the root-level readme file to the docs folder for now. Some of …

42b8670

…the content is specific to Parflow. We can re-use the content of this file later when the reorganization of the github repository is finalized.

Minor updates to the readme file.

a941ab6

Add a readme to describe both the workflow and the entire repository

3455c2c

Add an initial workflow chart for the hackathon activities

e605828

Update the environment setup flow to include a one-time setup step th…

ad590c2

…at supports both the core library configuration for the evaluation workflow/package and the runtime dependencies.

updated instruction to install deps

605a5f6

tested running notebooks with the new env set-up, data collection wor…

8ba7b8b

…ks fine. NWM and Parflow notebooks (#2) should be updated. They still point to cssi_evaluation.snow which no longer exists in our reorg branch

removed backup files b/c the updated env set-up workflow works fine

4785bda

merged cam-1157 and kept streamflow notebook but removed pixi.toml as…

f91a76c

… all dependencies is already mentioned in getting_started.md

Merge pull request #8 from CUAHSI/reorg

c0365e9

Merge reorg branch into develop branch

Merge pull request #9 from CUAHSI/develop

25a07cf

Merge develop branch into main branch

igarousi requested a review from amy-defnet March 17, 2026 12:02

amy-defnet mentioned this pull request Mar 17, 2026

Merge main CSSI repo back into fork CUAHSI/cssi_evaluation#10

Merged

Merge remote-tracking branch 'upstream/main'

95d16a3

Merged to be synced with the upstream.

amy-defnet requested changes Mar 17, 2026

View reviewed changes

pyproject.toml Outdated Show resolved Hide resolved

cssi_env.yml Outdated Show resolved Hide resolved

pyproject.toml Show resolved Hide resolved

revised deps based on Amy's feedback

65e3b80

amy-defnet approved these changes Mar 18, 2026

View reviewed changes

amy-defnet merged commit 4491c6f into hydroframe:main Mar 18, 2026
1 check failed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Merge CUAHSI/cssi_evaluation to hydroframe/cssi_evaluation#17

Merge CUAHSI/cssi_evaluation to hydroframe/cssi_evaluation#17
amy-defnet merged 75 commits intohydroframe:mainfrom
CUAHSI:main

igarousi commented Mar 17, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

amy-defnet left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

igarousi commented Mar 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

amy-defnet left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

igarousi commented Mar 17, 2026 •

edited

Loading