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

Document MODIS data sampling #311

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

Document MODIS data sampling #311

wants to merge 1 commit into from

Conversation

yellowcap
Copy link
Member

Added Modis to our training data set using stacchip.

  • Chip dates and locations data added to general index using the new modis processor in stacchip
  • All data prechipped using the slightly adapted prechp module
  • Stats computed using 10% of the data through the almost unchanged stats module

Most of the work went into finding a reasonable projection for the data, which comes in SIN grid originally. The processor reprojects the modis tiles into web mercator. The prechipping and stats were working almost out of the box 🚀 .

@yellowcap yellowcap requested a review from srmsoumya July 28, 2024 22:16
@brunosan
Copy link
Member

@yellowcap can you share more about the projection choice?
From what I remember we do not reproject in other cases, correct?

@srmsoumya
Copy link
Collaborator

@brunosan Based on my discussion with Daniel, default projection of MODIS had a lot of empty pixels, so he had to reproject them into Web Mercator & slice to areas that are best available.
Yes, we don't reproject any other data source.

Comment on lines +214 to +221
wavelength:
sur_refl_b01: 645.0
sur_refl_b02: 858.5
sur_refl_b03: 469.0
sur_refl_b04: 555.0
sur_refl_b05: 1240.0
sur_refl_b06: 1640.0
sur_refl_b07: 2130.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pass the wavelengths in millimeter:

Suggested change
wavelength:
sur_refl_b01: 645.0
sur_refl_b02: 858.5
sur_refl_b03: 469.0
sur_refl_b04: 555.0
sur_refl_b05: 1240.0
sur_refl_b06: 1640.0
sur_refl_b07: 2130.0
wavelength:
sur_refl_b01: .645
sur_refl_b02: .858
sur_refl_b03: .469
sur_refl_b04: .555
sur_refl_b05: 1.240
sur_refl_b06: 1.640
sur_refl_b07: 2.130

Comment on lines +195 to +196
- sur_refl_b07
gsd: 500
Copy link
Collaborator

Choose a reason for hiding this comment

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

We would also need an rgb index.

Suggested change
- sur_refl_b07
gsd: 500
rgb_indices:
- 0
- 3
- 2

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