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

Add CheckDataMixin #735

Open
wants to merge 5 commits into
base: gh/fzimmermann89/77/head
Choose a base branch
from

Conversation

fzimmermann89
Copy link
Member

@fzimmermann89 fzimmermann89 commented Mar 16, 2025

No description provided.

[ghstack-poisoned]
Copy link
Contributor

github-actions bot commented Mar 16, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/mrpro
   _version.py6267%7–8
src/mrpro/algorithms/csm
   inati.py24196%44
   walsh.py16194%53
src/mrpro/algorithms/dcf
   dcf_voronoi.py55493%15, 55–56, 89
src/mrpro/algorithms/optimizers
   adam.py22195%104
   pdhg.py81396%178–179, 185
   pgd.py53492%107, 152–155
src/mrpro/algorithms/reconstruction
   DirectReconstruction.py281643%59–79, 93
   IterativeSENSEReconstruction.py13192%79
   Reconstruction.py502256%42, 54–56, 80–87, 108–117
   RegularizedIterativeSENSEReconstruction.py411759%97–101, 115–140
src/mrpro/data
   AcqInfo.py141596%42, 120–121, 123, 229
   CheckDataMixin.py3313789%90, 96, 98, 110, 117, 125, 127, 129, 162–163, 186, 195, 212, 240, 256, 304, 309, 315–320, 350, 357, 402, 417, 496, 503, 519, 604, 626–630, 662, 703–706, 733
   CsmData.py29390%14, 83–85
   Dataclass.py2062886%39, 256, 272, 338, 396–398, 411, 493, 513–514, 516, 531–532, 534, 563–570, 576–581
   DcfData.py35294%15, 62
   EncodingLimits.py81396%33, 123, 126
   IData.py52296%119, 133
   IHeader.py1341291%69–72, 251, 255, 259, 263, 297–301
   KData.py2032986%113–114, 129, 136, 147–157, 166, 174, 183, 217, 239–241, 285–286, 349–360, 489, 491, 561
   KHeader.py1402185%23, 107–113, 140, 188, 195–196, 199, 206, 223–230, 238–249
   KNoise.py22959%37–50
   KTrajectory.py102397%164, 166, 186
   QData.py32197%43
   Rotation.py7313695%103, 201, 338, 436, 480, 498, 585, 587, 596, 630, 632, 695, 772, 777, 780, 795, 812, 817, 893, 1081, 1086, 1089, 1113, 1117, 1245, 1247, 1255–1256, 1320, 1402, 1647, 1730, 1882, 1917, 1921, 2111
   SpatialDimension.py2291892%34, 103, 150, 156, 276–278, 291–293, 327, 345, 358, 371, 384, 397, 406–407
   acq_filters.py12192%47
src/mrpro/data/traj_calculators
   KTrajectoryCalculator.py27196%84
   KTrajectoryIsmrmrd.py19195%57
src/mrpro/operators
   CartesianSamplingOp.py96496%124, 163, 238, 305
   ConstraintsOp.py60297%46, 48
   EndomorphOperator.py32294%52, 58
   FiniteDifferenceOp.py27293%40, 105
   FourierOp.py90397%185, 257, 262
   Functional.py77988%20–22, 117, 119, 226–228, 242
   GridSamplingOp.py136993%71–72, 81–82, 89–90, 93, 95, 97
   LinearOperator.py201896%107, 217, 244, 251, 292, 301, 309, 326
   LinearOperatorMatrix.py1621988%82, 119, 152, 161, 166, 175–178, 191–194, 202–203, 208–209, 221, 310, 337, 364
   MultiIdentityOp.py13285%43, 48
   NonUniformFastFourierOp.py1881095%69, 96, 206, 208, 241, 243, 319, 373, 423, 428
   Operator.py79297%32, 88
   ProximableFunctionalSeparableSum.py39392%50, 103, 110
   SliceProjectionOp.py174895%45, 62, 64, 70, 206, 227, 260, 300
   WaveletOp.py119596%151, 169, 204, 209, 232
   ZeroPadOp.py16194%30
src/mrpro/phantoms
   brainweb.py2753189%325–335, 371, 427–430, 452–453, 458–459, 461–462, 466, 474, 481–482, 520, 586–587, 606–609, 620, 622, 657–658, 671
src/mrpro/utils
   filters.py62297%44, 49
   indexing.py183199%320
   reshape.py95298%110, 301
   slice_profiles.py47687%21, 37, 116–119, 152
   sliding_window.py34197%34
   split_idx.py10280%43, 47
   summarize_tensorvalues.py12375%23, 25, 27
   typing.py735623%9–236
   unit_conversion.py601477%32, 40, 42, 49, 51, 58, 60, 69, 80, 82, 101, 103, 124, 126
   zero_pad_or_crop.py31681%26, 30, 55, 58, 61, 64
TOTAL628849792% 

Tests Skipped Failures Errors Time
2774 0 💤 0 ❌ 0 🔥 1m 43s ⏱️

[ghstack-poisoned]
fzimmermann89 added a commit that referenced this pull request Mar 17, 2025
ghstack-source-id: 17e796ab073bc2a810caddc8f752788b4769489a
ghstack-comment-id: 2727653597
Pull Request resolved: #735
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
Copy link
Contributor

📚 Documentation

📁 Download as zip
🔍 View online

@fzimmermann89 fzimmermann89 marked this pull request as ready for review March 18, 2025 22:42
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