Reduce memory usages - Githubissues

opera-adt / DSWX-SAR

Dynamic Surface Water Extent from Synthetic Aperture Radar

Apache License 2.0

9 stars 6 forks source link

Reduce memory usages #44

Closed oberonia78 closed 6 months ago

oberonia78 commented 7 months ago

This PR introduces the block processing for the steps consuming the memory.

[x] pre-processing
[x] initial_threshold
[x] fuzzy value computation
[x] region growing
[x] masking with ancillary data
[x] refinement using bimodality
[x] inundated vegetation

Previous memory usage:

Until now, all steps except masking with ancillary data was optimized and the memory usage has been reduced.

oberonia78 commented 7 months ago

@seongsujeong I think all functions are ready to review. Would you review it one more time?

oberonia78 commented 6 months ago

Thanks for the comments @seongsujeong @LiangJYu . I just finished the modification based on your comments. Would you review them one more time?

seongsujeong commented 6 months ago

@oberonia78 If the feature implemented in this PR covers the PR #39, then please consider closing the old one. Thanks.

oberonia78 commented 6 months ago

Thanks for the comments, @LiangJYu and @seongsujeong . I think I fixed the issue @LiangJYu mentioned. I believe it is ready to go. If there is no concern, I'll merge this PR.