search

opera-adt / DSWX-SAR

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

Fix warnings #36

Closed oberonia78 closed 9 months ago

oberonia78 commented 9 months ago

This PR fixes the lines that may raise the warning message.

Some variables may have zero-value and cause the warning message when division is applied. To avoid this, this PR replaced value with very small value (i.e. 1e-10).

In addition, the warning message sometime occurred when the bursts only cover the water (ocean). So, to avoid this, the spatial water coverage is introduced, and the initial thresholds are computed only when the land and water coexists.

oberonia78 commented 9 months ago

@seongsujeong Just fixed the lines you mentioned. Thank you for the comments.

oberonia78 commented 9 months ago

Finished the modification. Would you have another look? @seongsujeong