ASFHyP3 / hyp3-isce2

HyP3 plugin for ISCE2 processing
Apache License 2.0
11 stars 9 forks source link

add description of water mask option in the README.md and update teh … #200

Closed cirrusasf closed 7 months ago

cirrusasf commented 7 months ago

…CHANGELOG.md

github-actions[bot] commented 7 months ago

Coverage

Coverage Report
FileStmtsMissCoverMissing
/home/runner/micromamba/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2
   __main__.py26292%33, 44
   burst.py2505578%98–101, 114–119, 133–148, 164–173, 187–197, 255, 303–311, 331–345, 583–585, 611
   dem.py53394%101–110
   insar_stripmap.py694239%38–91, 95–97, 114–133
   insar_tops.py573047%49–87, 110–132
   insar_tops_burst.py23217823%91–163, 175–201, 231–314, 326–328, 341–439, 463–479, 515–599
   logging.py4175%9
   merge_tops_bursts.py6289585%184, 201, 263, 268, 325, 341, 349, 381, 385, 471, 498, 576, 616–634, 683–691, 715, 720, 779, 802, 946, 1129–1149, 1164–1177, 1191–1217, 1228–1231, 1236–1249, 1253
   s1_auxcal.py21290%51, 59
   slc.py341071%22–25, 29–34
   stripmapapp_alos.py50786%102, 138, 141–146
   topsapp.py70889%77, 107, 129–135
   utils.py2101095%114–125, 130, 236, 252–254, 261, 401, 446
   water_mask.py59592%88, 100–103
TOTAL178944875% 

Tests Skipped Failures Errors Time
85 0 :zzz: 0 :x: 0 :fire: 30.445s :stopwatch:
forrestfwilliams commented 7 months ago

Honestly I don't think we even need this PR. I think adding the extra info in the readme is kinda clunky and everything else is already fixed!

cirrusasf commented 7 months ago

I agree with your idea. We don't need this PR.