Skip to content

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 jsreu January 13, 2025 13:56
@jsreu
Copy link
Collaborator

jsreu 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

@jsreu
Copy link
Collaborator

jsreu 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.

3 participants