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

ALOS OALSAR1 SICD Format Support #539

Open
syu-tan opened this issue Aug 29, 2024 · 3 comments
Open

ALOS OALSAR1 SICD Format Support #539

syu-tan opened this issue Aug 29, 2024 · 3 comments
Labels
question Further information is requested

Comments

@syu-tan
Copy link

syu-tan commented Aug 29, 2024

Thank you for the great SAR repository.

I would like to apply the SICD format for ALOS PALSAR1.
I have confirmed it with subaperture. I would be happy if i pull request. I would love to contribute.
000_CSI_1_plot_full

I don't have permission when pushing, so I get a 403 error.
Please grant me permission.

スクリーンショット 2024-08-30 0 12 56
@pressler-vsc pressler-vsc added the question Further information is requested label Aug 29, 2024
@pressler-vsc
Copy link
Collaborator

@syu-tan, thanks for the offer! SARPy uses the fork and pull request model for contributions, which means no additional permissions are necessary.

There's some information regarding SARPy contributions scattered in a few READMEs (apologies for the current scattered nature of the "Contributing" documentation...):

Please reach out with any further questions and we look forward to your contribution!

@utwade
Copy link
Contributor

utwade commented Aug 29, 2024

Note that there is one irreconcilable difference between the current SICD spec and ALOS1, as I understand it. What I have been told by others who know more than me about that sensor is that ALOS1 data is in a native Doppler geometry, which isn't really a grid currently supported by SICD (as opposed to a zero Doppler geometry that is used in SICD and many modern sensors). That said, it may still effectively be able to get you most of the way there for much of the functionality you may be looking for (as you have shown in your results.) It is possible that geolocation will be a bit off though.

@syu-tan
Copy link
Author

syu-tan commented Aug 29, 2024

Thank you for sharing this important information.
The distortion of iso doppler has not been corrected. I haven't done the calculations, but I think that if the difference in squint angle is large, the change in the azimuth direction cannot be reproduced in the SICD format.
Still, I thought the attempt to unify it to NGA was great.

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

No branches or pull requests

3 participants