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.

Sign up for GitHub

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

Jump to bottom

Add Landsat 8/9 tasseled cap coefficients #42

Closed
aazuspan opened this issue Mar 23, 2022 · 4 comments · Fixed by #43
Closed

Add Landsat 8/9 tasseled cap coefficients #42

aazuspan opened this issue Mar 23, 2022 · 4 comments · Fixed by #43
Assignees
@aazuspan
Labels
enhancement New feature or request

Comments

@aazuspan
Copy link
Collaborator

Tasseled cap coefficients for Landsat 8/9 TOA and SR data were recently published by Zhai et al. We already have Landsat 8 coefficients from Baig et al, so I'll need to do some reading to see which to include...
@aazuspan aazuspan added the enhancement New feature or request label Mar 23, 2022
@aazuspan aazuspan self-assigned this Mar 23, 2022
@davemlz
Copy link
Collaborator

davemlz commented Mar 23, 2022

Hey @aazuspan!

That's nice! 🚀 For L9 the coefs should be the same as L8, right?

PD: I also added L9 for the remaining features, like spectralIndices and maskClouds. However, Landsat 9 is not yet included in the GEE STAC (I don't know why) and the tests are broken. I will remove L9 from the tests for now and when L9 is added I'll also do it :)

Best,

David

@aazuspan
Copy link
Collaborator Author

Hey @davemlz, yep, the L8 coeffs should work for L9, at least until something gets published specifically based on L9 data :)

Thanks for the heads up on the STAC issue! I'll make sure to disable the L9 tasseled cap tests if I make a PR before that's added.

Can't wait to have all the features up and running for L9!

aazuspan added a commit to aazuspan/ee_extra that referenced this issue Mar 25, 2022
@aazuspan
Add TC coefficients for L8 and L9, SR and TOA (r-earthengine#42)
216917b
aazuspan added a commit to aazuspan/ee_extra that referenced this issue Mar 25, 2022
@aazuspan
Add TC tests for L8 SR and L9 TOA and SR (r-earthengine#42)
8cdbac6
@aazuspan aazuspan mentioned this issue Mar 25, 2022
Closed
@davemlz
Copy link
Collaborator

davemlz commented Mar 30, 2022

Hey @aazuspan, just to let you know that L9 is now in the STAC! 🚀

@aazuspan
Copy link
Collaborator Author

Awesome, thanks @davemlz! PR incoming :)

aazuspan added a commit to aazuspan/ee_extra that referenced this issue Mar 31, 2022
@aazuspan
Remove L9 RT and T2 collections (r-earthengine#42)
66a8a59
@aazuspan aazuspan mentioned this issue Mar 31, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aazuspan aazuspan

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Landsat 8/9 tasseled cap coefficients aazuspan/ee_extra
2 participants
@davemlz @aazuspan