opera-adt / DSWX-SAR

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

debug remaining issues #66

Closed oberonia78 closed 7 months ago

oberonia78 commented 7 months ago

This PR fixes the issues in gamma delivery. 1) Reduce the memory in bimodality and masking procedures 2) Fixes the bug in the initial threshold when the multi otsu thresholding. 3) Fixes the bug in writing the block when the block is 1 line.

oberonia78 commented 7 months ago

@LiangJYu @seongsujeong Thanks you for your comments. I fixed all based on your comments. Would you have another look?

oberonia78 commented 7 months ago

Probably, you will see many changes from the last commit. But many of them are just reformatting the long line or missing spaces.

oberonia78 commented 7 months ago

@LiangJYu @seongsujeong I just finished the correction based on the second-round review. Please have another look. Thanks all.