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

Slice thickness test xleyn #460

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Slice thickness test xleyn #460

wants to merge 3 commits into from

Conversation

xleyn
Copy link
Collaborator

@xleyn xleyn commented Jan 6, 2025

Resubmitting previous pull requests from a branch rather than from a local fork, as requested by Molly.

Main changes are:

  1. Function added in ACRObject.py to accurately detect the angle of the phantom's rotation.
  2. Functions in acr_slice_thickness.py to place two lines within the central insert of the phantom and calculate profiles across them. The lines are still placed accurately even when the phantom is rotated.

Copy link

github-actions bot commented Jan 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
hazenlib
   ACRObject.py1272779%48, 82–88, 108–130, 159, 174–177, 205–206, 244–247
   HazenTask.py29390%67–71
   __init__.py641773%104, 142–151, 153–162, 164–166, 185–191, 195
   exceptions.py21576%19–23, 42
   utils.py2406274%90, 94, 115, 128, 161, 176–189, 208, 215–222, 239, 246–248, 263–267, 283–287, 301, 326, 331, 342, 360, 422–423, 425–426, 431–456, 509, 512, 520–525, 528, 583, 592, 621
hazenlib/tasks
   acr_geometric_accuracy.py1115848%53–100, 124–239
   acr_ghosting.py1064260%42–58, 104–107, 154–157, 201–283
   acr_slice_position.py1364865%56–80, 281–348
   acr_slice_thickness.py1698749%47–79, 178, 256–340, 357–361, 374–412
   acr_snr.py1325757%60–111, 131, 227–242, 287–305, 355–380
   acr_spatial_resolution.py2066867%70–100, 187, 285, 302–313, 460–539
   acr_uniformity.py793259%43–60, 130–182
   ghosting.py1485265%28–47, 67, 168–169, 176, 193–194, 249–253, 268–272, 343–385
   relaxometry.py2918969%210–211, 213, 226–231, 238–246, 277–326, 375, 409–431, 609, 655–659, 726, 811–833, 851–866
   slice_position.py1244068%30, 43–71, 129–130, 157, 273, 283–306
   slice_width.py3525285%44–48, 52, 123, 188–213, 555, 560–561, 567, 572, 648–649, 1020–1084
   snr.py1736960%45–48, 87, 103–113, 206–225, 237–247, 287–302, 328–338, 343–359, 397–413, 426–432, 475–493
   snr_map.py108199%159
   spatial_resolution.py2464482%50–54, 58, 90, 213, 294, 460–501
   uniformity.py791976%59–63, 67, 118–119, 126, 174–204
TOTAL296687271% 

Tests Skipped Failures Errors Time
205 0 💤 0 ❌ 0 🔥 4m 20s ⏱️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant