Skip to content

update zonal stats example #350

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

Merged
merged 3 commits into from
Jan 23, 2025
Merged

update zonal stats example #350

merged 3 commits into from
Jan 23, 2025

Conversation

sterfd
Copy link
Contributor

@sterfd sterfd commented Jan 23, 2025

No description provided.

@MaxLenormand
Copy link
Collaborator

FYI the CI pipeline is breaking because you changed a link and it's not pointing to the write place anymore

We don't want to ship changes with broken links, this is how we catch it :)

Copy link
Collaborator

@MaxLenormand MaxLenormand left a comment

Choose a reason for hiding this comment

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

This is nice! I'll take another look once the CI passes so I can check on staging

@sterfd sterfd force-pushed the example_zonal_stats_updates branch from bf54e92 to b7cf17f Compare January 23, 2025 15:06
@MaxLenormand
Copy link
Collaborator

Requirements for both examples are different now 🙃

CleanShot 2025-01-23 at 16 55 07

CleanShot 2025-01-23 at 16 55 41

Either one (:::note or ###Requirements) is good for me, but can you please make them similar

@sterfd sterfd merged commit fb147c2 into main Jan 23, 2025
1 check passed
@sterfd sterfd deleted the example_zonal_stats_updates branch January 23, 2025 20:53
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