Precommit: run mypy on all files, including tests.

[fzimmermann89]

Runs mypy on all files. As changes to one file can cause typing issues in unchanged files.

[fzimmermann89]

Stack from ghstack (oldest at bottom):

• -> Precommit: run mypy on all files, including tests. #283
• Fix types #282
• Add EndomorphOperator #280
• Unify .to / .cpu / .cuda for data classes #279

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
src/mrpro/algorithms/csm
_iterative_walsh.py	15	1	93%	34
src/mrpro/algorithms/optimizers
_adam.py	17	1	94%	82
src/mrpro/algorithms/reconstruction
_direct_reconstruction.py	63	36	43%	53–58, 75–83, 95–97, 113–120, 134–145
_reconstruction.py	10	1	90%	20
src/mrpro/data
_AcqInfo.py	83	2	98%	100, 138
_CsmData.py	21	3	86%	28, 72–74
_DcfData.py	91	5	95%	36, 42, 86, 111, 199
_IData.py	61	3	95%	136, 142, 146
_IHeader.py	59	2	97%	80, 107
_KHeader.py	127	9	93%	37, 84, 122, 167, 178, 185–186, 189, 196
_KNoise.py	22	9	59%	54–67
_MoveDataMixin.py	133	14	89%	36, 130, 146, 160–162, 223, 286, 300, 379, 399–400, 417–418
_QData.py	31	1	97%	56
_SpatialDimension.py	46	1	98%	80
_TrajectoryDescription.py	14	1	93%	38
src/mrpro/data/_kdata
_KData.py	108	7	94%	104–105, 175–176, 218, 223–224
_KDataRemoveOsMixin.py	30	2	93%	53, 55
_KDataSelectMixin.py	22	2	91%	62, 78
_KDataSplitMixin.py	51	3	94%	67, 97, 106
src/mrpro/data/traj_calculators
_KTrajectoryCalculator.py	27	2	93%	40, 62
_KTrajectoryIsmrmrd.py	16	2	88%	58, 67
_KTrajectoryPulseq.py	30	1	97%	69
src/mrpro/operators
_CartesianSamplingOp.py	50	9	82%	63–64, 69–70, 75–76, 102, 105, 128
_ConstraintsOp.py	61	2	97%	38, 40
_EndomorphOperator.py	57	3	95%	219, 223, 227
_FastFourierOp.py	31	2	94%	84, 94
_FourierOp.py	78	1	99%	146
_GridSamplingOp.py	123	9	93%	74–75, 84–85, 92–93, 96, 98, 100
_LinearOperator.py	64	3	95%	41, 142, 147
_Operator.py	56	1	98%	38
_SliceProjectionOp.py	168	8	95%	55, 62, 64, 70, 192, 213, 246, 286
_ZeroPadOp.py	16	1	94%	44
src/mrpro/utils
_Rotation.py	457	30	93%	60–70, 110, 287, 373, 375, 388, 445, 450, 453, 468, 484, 488, 633, 638, 641, 656, 659, 734, 736, 744–745, 985, 1067
_split_idx.py	10	2	80%	57, 61
_zero_pad_or_crop.py	31	6	81%	40, 44, 68, 71, 74, 77
filters.py	68	6	91%	48–50, 75, 80, 87
slice_profiles.py	45	6	87%	31, 47, 124–127, 160
sliding_window.py	35	1	97%	47
TOTAL	3207	198	94%

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

[fzimmermann89]

As changes to a file can also lead to type errors in other files, only checking committed files does not help.
Also, a lot of non-commited files will be parsed by mypy anyways, if they are imported. Overall, mypy suggested to run on all files
python/mypy#13916

As #282 fixes all remaining issues in the tests, we can now also check the tests.

[fzimmermann89]

Mypy cannot infer that the nullcontext or pytest.warns condition always results in a context manager.

This either needs a type hint or a different logic.

python/mypy#10109

--> fixed in #282

[fzimmermann89]

The only issue now is, that if you have some non-staged files with typing issues in the src directory,
the precommit will fail...

not sure if there is an easy way to run on 'staged and already committed files' in precommit..