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

Satellite image band list #23

Closed
MWieland opened this issue May 13, 2020 · 11 comments
Closed

Satellite image band list #23

MWieland opened this issue May 13, 2020 · 11 comments
Assignees
Labels
enhancement New feature or request question Further information is requested
Milestone

Comments

@MWieland
Copy link
Collaborator

For any image preprocessing and analysis steps we should be able to store and keep track of the band information of a satellite image. In particular, we should know which band index refers to which wavelength and which unit (e.g. digital numbers, top of atmosphere, sigma naught, etc.). This is essential to be able to compute even the simplest band indices (e.g. NDVI) or to select an appropriate band subset during preprocessing.

We have already a dedicated metadata attribute for this ("bandlist") but do not fill it upon metadata creation, because this information is not directly provided by the source metadata (e.g. EarthExplorer). It could like the following for a band subset of a Sentinel-2 image:

'bandlist': [{'index':0, 'name':'B02', 'wavelength':'blue', 'unit':'DN'}, {'index':1, 'name':'B03', 'wavelength':'green', 'unit':'DN'}, {'index':2, 'name':'B04', 'wavelength':'red', 'unit':'DN'}, {'index':3, 'name':'B08', 'wavelength':'nir', 'unit':'DN'}, {'index':4, 'name':'B11', 'wavelength':'swir1', 'unit':'DN'}, {'index':5, 'name':'B12', 'wavelength':'swir2', 'unit':'DN'}]

We could compile this ourselves from a bunch of lookup tables for unique combinations of "Platform" and "Producttype". We basically do this already here for Landsat.

This should be put in a public place (possibly here) and be expanded upon. Then we can compile the respective bandlist "on the fly" when we query the satellite image metadata. This would also allow a user to simply update it in a standardized way later on as part of a more complex processing chain.

@MWieland MWieland added enhancement New feature or request question Further information is requested labels May 13, 2020
@fwfichtner
Copy link
Contributor

You could solve this with enums again.

@MWieland
Copy link
Collaborator Author

Good idea. Will give it a try. Thanks!

@fwfichtner fwfichtner added this to the 0.5.0 milestone Jun 10, 2020
@fwfichtner
Copy link
Contributor

Do we address this for 0.5.0 or shall we release?

@MWieland
Copy link
Collaborator Author

Would be great to put this into this release. Will try to find time this week to work on it. Deadline Friday 3 July alright?

@MWieland
Copy link
Collaborator Author

MWieland commented Jul 2, 2020

Let's move it to the next release. I simply can't find enough time these days to dedicate to develop it.

@fwfichtner fwfichtner modified the milestones: 0.5.0, 0.6.0 Jul 3, 2020
@fwfichtner fwfichtner removed this from the 0.6.0 milestone Aug 12, 2020
@MWieland
Copy link
Collaborator Author

MWieland commented Mar 9, 2021

This will be fixed as part of #98 once we have implemented the sensor-specific STAC standard suggestions

@fwfichtner fwfichtner added this to the 1.2.0 milestone Mar 23, 2021
@fwfichtner fwfichtner modified the milestones: 1.2.0, 1.3.0 Apr 7, 2021
@fwfichtner
Copy link
Contributor

@MWieland what is the status on this?

@MWieland
Copy link
Collaborator Author

This is not resolved yet. Moving an updated roadmap from #98 to here for completeness:

Roadmap to move ukis_pysat.data to STAC

  • Return metadata as pystac Items.
  • Develop our own STAC-API search class (inspired by sat-search).
  • Convert metadata items returned by data providers dynamically to STAC Items according to best practises for different platforms. Currently we do this hardcoded and only for a small subset of attributes. The full set of attributes should also include the image band list.

@fwfichtner
Copy link
Contributor

@shiwakotisurendra can you take care of that?

@fwfichtner
Copy link
Contributor

@MWieland does #144 fix this?

@fwfichtner
Copy link
Contributor

We will try to follow best practices more in the future -> #155

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants