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

Creating COG writing and validation docs #382

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

omshinde
Copy link
Contributor

@omshinde omshinde commented Feb 2, 2024

This PR consists of documentations for-

  1. Writing a (Cloud-Optimized GeoTIFF) COG output with Python
  2. Validating the COG vs Non-COG

I have a few queries about the location for these documentations.

Reason for choosing ASTER_L1T dataset:

  • Ongoing mission
  • It has two OnlineHTTPAccessURLs and we don't have any example in MAAP Documentation for using one URL out of the two OnlineHTTPAccessURLs. So, it can act as a reference for such use-case.

cc: @wildintellect @smk0033

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@wildintellect
Copy link
Collaborator

@omshinde the Guide

  • doesn't show how to properly make a COG using python code
  • doesn't show how to validate the data inside the COG

I actually think using something with 2 urls make's it confusing, how to select between 2 urls should be it's own technical topic in the CMR/STAC usage section.

Most of what you wrote could be converted to a science tutorial about accessing ASTER-1

@omshinde
Copy link
Contributor Author

omshinde commented Feb 2, 2024

Thank you @wildintellect ..
As suggested, I'll keep these notebooks for accessing ASTER-1 and prepare another for the guide. Do you have any recommendations on the data for the guide tutorial? Is this the one we discussed last time- NASA Shuttle Radar Topography Mission Global 1 arc second NetCDF V003 on NASA CMR.

@wildintellect
Copy link
Collaborator

Might as well use the updated dataset NASADEM https://search.earthdata.nasa.gov/search/granules?p=C1546314043-LPDAAC_ECS&pg[0][v]=f&pg[0][gsk]=-start_date&q=NASADEM&tl=1706908327.71!3!!

@@ -0,0 +1,486 @@
{
Copy link
Contributor

@smk0033 smk0033 Feb 5, 2024

Choose a reason for hiding this comment

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

My only comment is doing mixed case on the headers for both tutorials, like "Accessing the Dataset". Other than that, I think the formatting looks good!

Just a side note, but you can probably remove my name from the authors on both tutorials - I appreciate it, but definitely haven't contributed enough to these


Reply via ReviewNB

@@ -0,0 +1,411 @@
{
Copy link
Contributor

@smk0033 smk0033 Feb 5, 2024

Choose a reason for hiding this comment

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

One other note is this should be at least a ## header if not smaller (should definitely be smaller than the title). Sorry if it is smaller than the title, not sure if it's the way it's showing up on here or not


Reply via ReviewNB

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.

3 participants