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

Implemented preprocessing steps for Sentinel-1 data using esa-snappy #62

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

katyagikalo
Copy link
Collaborator

@katyagikalo katyagikalo commented Jan 13, 2025

Following preprocessing steps were added:

  • Orbit file correction
  • Thermal noise removal
  • Calibration
  • Speckle filtering
  • Terrain correction

Closes #61

@katyagikalo katyagikalo requested a review from jsreuss January 13, 2025 13:56
@jsreuss
Copy link
Collaborator

jsreuss commented Jan 13, 2025

@katyagikalo : acquisition/clipping/calibration.py can be removed I think. We're not using it anymore, are we?

@katyagikalo
Copy link
Collaborator Author

@katyagikalo : acquisition/clipping/calibration.py can be removed I think. We're not using it anymore, are we?

Yes, I deleted it

@jsreuss
Copy link
Collaborator

jsreuss commented Jan 14, 2025

TODOs:

  1. Move all esa_snappy imports into separate module s.t. it does not have to be installed if not used.
  2. Finish installation guide for esa_snappy in ReadME (macOS user still missing)
  3. Add variable to config for choosing DEM (cf. s1_preprocessing.terrain_correction for available values)
  4. (Optional) Add bool-variables to config s.t. the user can decide which pre-processing steps he wants to perform. Keep the defaults as True s.t. it performs the ones you mentioned by default.

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.

2 participants