opera-adt / DSWX-SAR

Dynamic Surface Water Extent from Synthetic Aperture Radar
Apache License 2.0
10 stars 6 forks source link

add preprocessing step #12

Closed oberonia78 closed 1 year ago

oberonia78 commented 1 year ago

This PR adds the preprocessing step into the DSWx-SAR workflow. The preprocessing includes

oberonia78 commented 1 year ago

@seongsujeong Thank you for your comments. It seems it requires more clearing up. I just finished fixing the code based on your comments. For your information, dummpy_dswx_s1.py will be replaced with the new files. So, I did not modify the code for this.

seongsujeong commented 1 year ago

@oberonia78 Thanks for the information. I'll disregard dummpy_dswx_s1.py when reviewing the PR.

oberonia78 commented 1 year ago

@seongsujeong I revised the PR based on the comments. Let me know if you have any comments. If not, let's merge it ASAP.

oberonia78 commented 1 year ago

@seongsujeong All bugs were corrected and docstrings were added. Let me know if you have more comments.

seongsujeong commented 1 year ago

@oberonia78 one more comment here Please make sure to check that out before merging this PR.