Skip to content

Add ESA CCI V4 notebook#257

Merged
smk0033 merged 9 commits intodevelopfrom
add-esa-cci-v4-notebook
Oct 2, 2023
Merged

Add ESA CCI V4 notebook#257
smk0033 merged 9 commits intodevelopfrom
add-esa-cci-v4-notebook

Conversation

@emileten
Copy link
Contributor

@emileten emileten commented Jul 4, 2023

Adding a notebook that demonstrates how to access and visualize the ESA CCI V4 collection from the MAAP STAC catalog. This follows a user question about how to access this data..

In this notebook I turn a collection search into an array using stackstac and plot a time series of the average AGB over a single tile. I also show how to access the catalog field that contains the storage location information.

@nehahunka do you think this would be useful ? I am wondering if a different, or richer, analysis would be more helpful.

@wildintellect tagging you to get your opinion about whether this is useful. Note that this is a draft PR, there are definitely things to improve, notably standardize the notebook to our standards in terms of format and documentation, and if possible avoid requiring to create an extra environment (somehow for now I couldn't make it work without doing this).

@emileten emileten self-assigned this Jul 4, 2023
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@wildintellect
Copy link
Collaborator

wildintellect commented Jul 5, 2023

@emileten I think it's useful, would suggest adding a visualization of maybe 2 time periods side by side for some portion of the cube.

@smk0033 can review for standardization.

@wildintellect wildintellect requested a review from smk0033 July 5, 2023 16:36
Copy link
Contributor

@smk0033 smk0033 left a comment

Choose a reason for hiding this comment

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

Looks good overall! I added some reviews in ReviewNB - let me know if you can't see them. Mostly just dealing with additional information in the beginning (at the very least, I'd add author(s), date and change "What are we doing in this notebook?" to "Description" and remove it as a header). You can add other sections I mentioned in ReviewNB if you wish, but I can also go in and add them a little later on. From there, I'd also add some other headers to help generalize sections

@nehahunka
Copy link

nehahunka commented Jul 5, 2023 via email

@emileten
Copy link
Contributor Author

emileten commented Jul 7, 2023

Thanks @nehahunka ! What you mentioned should be easy to add. As per the notebook you mentioned, I asked @abarciauskas-bgse !

@emileten emileten changed the title Draft: add ESA CCI V4 notebook Add ESA CCI V4 notebook Oct 2, 2023
@emileten
Copy link
Contributor Author

emileten commented Oct 2, 2023

@smk0033 can you approve and merged this ? I can't do it myself since I authored this one.

I merged Rajat's changes (that you had approved). Thanks !

@smk0033 smk0033 merged commit 49acf49 into develop Oct 2, 2023
@smk0033 smk0033 deleted the add-esa-cci-v4-notebook branch October 2, 2023 13:33
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.

5 participants