Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update based on release review #246

Merged
merged 2 commits into from
Aug 26, 2024
Merged

Update based on release review #246

merged 2 commits into from
Aug 26, 2024

Conversation

forrestfwilliams
Copy link
Contributor

  • Update names of core topsapp functionality to remove "burst" qualifier
  • Mention moving of packaging utilities to new file in the changelog

@forrestfwilliams forrestfwilliams added the bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version label Aug 26, 2024
@forrestfwilliams forrestfwilliams requested a review from a team August 26, 2024 20:59
Copy link
Contributor

Coverage

Coverage Report
FileStmtsMissCoverMissing
/home/runner/micromamba/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2
   __main__.py26292%33, 44
   burst.py2505578%97–100, 113–118, 132–147, 163–172, 186–196, 254, 302–310, 330–344, 555–557, 583
   dem.py53394%101–110
   insar_stripmap.py755033%38–101, 105–107, 122–139
   insar_tops.py825533%47–97, 127–176, 198–216
   insar_tops_burst.py1359827%61–132, 143–197, 209–231, 235–237, 257–289
   logger.py4175%9
   merge_tops_bursts.py6299585%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
   packaging.py19914826%45–47, 69–93, 130–228, 248–264, 284–294, 306–334, 346–348, 377–466, 470–476
   s1_auxcal.py21290%51, 59
   slc.py462057%23–30, 34–39, 50–56
   stripmapapp_alos.py50786%102, 138, 141–146
   topsapp.py70889%77, 107, 129–135
   utils.py1921095%109–120, 125, 203, 219–221, 228, 368, 413
   water_mask.py59592%88, 100–103
TOTAL191255971% 

Tests Skipped Failures Errors Time
81 0 💤 0 ❌ 0 🔥 33.729s ⏱️

@forrestfwilliams forrestfwilliams merged commit 834e432 into develop Aug 26, 2024
9 checks passed
@forrestfwilliams forrestfwilliams deleted the review branch August 26, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants