Skip to content

Updated the CCSS notebook example based on the new structure#25

Merged
danielletijerina merged 1 commit intomainfrom
feature/update-ccss-notebook
Mar 25, 2026
Merged

Updated the CCSS notebook example based on the new structure#25
danielletijerina merged 1 commit intomainfrom
feature/update-ccss-notebook

Conversation

@igarousi
Copy link
Collaborator

Regarding Danielle's suggestion on replacing the map plot with the one Amy created, I believe plot_obs_locations() and plot_sites_within_domain() serve different purposes.

  • plot_obs_locations() creates a static plot of observation locations using model grid indices with imshow() and scatter(). It does not use geographic coordinates or shapefiles, and it saves the output as a figure.
  • In contrast, plot_sites_within_domain() uses geospatial coordinates, displays site markers over the watershed boundary, adds a basemap, and returns an interactive folium.Map object.
    Given these differences, I decided to keep plot_sites_within_domain() for this notebook, but I renamed the function name to map_sites_within_watershed() to better reflect its purpose.

Copy link
Collaborator

@danielletijerina danielletijerina 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! I like the CCSS photo addition!

@danielletijerina danielletijerina merged commit 10ac793 into main Mar 25, 2026
1 check passed
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