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

compact polarization #20

Open
pwong86 opened this issue Jan 31, 2025 · 5 comments
Open

compact polarization #20

pwong86 opened this issue Jan 31, 2025 · 5 comments

Comments

@pwong86
Copy link

pwong86 commented Jan 31, 2025

To support compact polarization, can we add LH, LV, RH and RV as possible values for sar:polarizations?

@m-mohr
Copy link
Contributor

m-mohr commented Feb 6, 2025

I guess we can, but I don't have enough knowledge about SAR to decide whether this makes sense.

@akerosenqvist Could you help me make the right decision here?
Does it make sense to add the values listed above to sar:polarization, which is currently defined here: https://github.com/stac-extensions/sar?tab=readme-ov-file#sarpolarizations

@akerosenqvist
Copy link

Yes, that makes sense. In addition, include also CH and CV.

@m-mohr
Copy link
Contributor

m-mohr commented Feb 7, 2025

Thanks!

@pwong86 Would you mind creating a PR for it?

@pwong86
Copy link
Author

pwong86 commented Feb 7, 2025

@m-mohr I can create a PR for it. Can you give me permission to push to this repo?

@m-mohr
Copy link
Contributor

m-mohr commented Feb 8, 2025

@pwong86 Thanks. Please fork the repository and create a PR from your fork.

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

No branches or pull requests

3 participants