386 Verbose option for slice position

[sophie22]

General improvement to the way of passing key word arguments from the CLI to the relevant HazenTask class (rather than to its .run() function. Add CLI option to slice position task to display the rounded positions for all 40 slices with --verbose

[github-actions[bot]]

Coverage Report

File	Stmts	Miss	Cover	Missing
hazenlib
ACRObject.py	107	11	90%	74, 77–79, 84–87, 125, 168–169
HazenTask.py	29	3	90%	41–43
__init__.py	57	15	74%	166, 197–200, 202–206, 208–210, 225–230, 234
exceptions.py	21	5	76%	17–21, 38
utils.py	189	43	77%	61, 65, 75, 80, 117, 124–129, 140, 143–150, 170–172, 190–192, 211–213, 222, 227, 233, 284, 287, 295–300, 303, 346, 355, 371
hazenlib/tasks
acr_geometric_accuracy.py	119	62	48%	42–86, 96–121, 134–168
acr_ghosting.py	107	42	61%	35–51, 77–79, 109–111, 145–187
acr_slice_position.py	136	48	65%	47–68, 194–241
acr_slice_thickness.py	136	60	56%	34–53, 162–224
acr_snr.py	136	58	57%	46, 51–92, 102, 172–182, 216–231, 264–281
acr_spatial_resolution.py	206	68	67%	59–86, 131, 174, 187–196, 278–330
acr_uniformity.py	81	32	60%	38–56, 112–138
ghosting.py	151	51	66%	19–36, 51, 113–114, 118, 128–129, 155–157, 174–176, 222–259
relaxometry.py	291	89	69%	207–208, 210, 221–224, 230–236, 268–312, 353, 387–405, 580, 626–628, 695, 769–791, 809–824
slice_position.py	124	40	68%	26, 31–62, 114–115, 141, 221, 229–247
slice_width.py	352	52	85%	36–38, 42, 107, 166–187, 441, 446–447, 453, 458, 516–517, 767–809
snr.py	168	67	60%	61, 78–83, 179–197, 212–221, 239–249, 276–286, 291–301, 332–346, 351–359, 388–402
snr_map.py	108	1	99%	152
spatial_resolution.py	249	44	82%	41–43, 47, 68, 153, 212, 338–375
uniformity.py	79	19	76%	47–49, 53, 95–96, 103, 137–152
TOTAL	2871	810	72%

Tests	Skipped	Failures	Errors	Time
201	0 :zzz:	0 :x:	0 :fire:	4m 0s :stopwatch:

[sophie22]

@tomaroberts ready for review small PR grew into a slim medium: changing the way CLI options are being passed to Task objects, primarily to enable adding the --verbose option to the slice position task as physicists requested, consequently updated the way SNR and ACR SNR tasks take their CLI args - relevant for @mollybuckley too! (happy to explain in detail)

[tomaroberts]

@sophie22 – this is ready to merge, however, there are a couple of merge conflicts which have popped up since I merged #381. Can you resolve and then we'll proceed.

[sophie22]

merge conflicts resolved, ready for final review @tomaroberts