Slice pos axis labels

[Lucrezia-Cester]

added axes labels to scatter figure for slice_position task, solves issue https://github.com/GSTT-CSC/hazen/issues/234

[github-actions[bot]]

Coverage Report

File	Stmts	Miss	Cover	Missing
hazenlib
HazenTask.py	24	3	88%	29–31
__init__.py	147	38	74%	136, 140, 150, 155, 192, 199–204, 215, 218–225, 245–247, 265–267, 286–288, 297, 302, 308, 359, 379–381, 389–390, 394
exceptions.py	21	4	81%	17–21
relaxometry.py	317	90	72%	238–256, 631, 690–692, 746, 794–816, 834–849, 1174–1177, 1186–1189, 1201–1214, 1217–1222, 1233–1263
shapes.py	20	9	55%	13, 16, 24–29, 32
snr_map.py	108	5	95%	403, 408–410, 439
tools.py	84	8	90%	43–50, 92, 101, 117
hazenlib/tasks
ghosting.py	150	51	66%	18–32, 47, 109–110, 114, 124–125, 151–153, 170–172, 218–256
relaxometry.py	7	2	71%	10–11
slice_position.py	118	22	81%	31, 40–41, 103–104, 130, 210, 217–234
slice_width.py	356	52	85%	34–37, 107, 166–186, 451, 456–457, 463, 468, 530–531, 780–821
snr.py	163	66	60%	62–67, 161–179, 194–203, 221–231, 258–268, 273–283, 314–327, 332–340, 369–382
snr_map.py	7	7	0%	1–11
spatial_resolution.py	246	44	82%	36–39, 62, 147, 206, 332–368
task_acr_ghosting.py	116	42	64%	33–53, 91–93, 123–125, 161–194
task_acr_uniformity.py	89	32	64%	34–54, 121–138
uniformity.py	78	19	76%	42–45, 91–92, 99, 133–147
TOTAL	2067	494	76%

Tests	Skipped	Failures	Errors	Time
180	0 :zzz:	0 :x:	0 :fire:	2m 19s :stopwatch:

[Lucrezia-Cester]

Hi @heyhaleema, @laurencejackson, could you please review this PR? Thank you.

[Lucrezia-Cester]

Hi @heyhaleema, could you review this PR please?

[Lucrezia-Cester]

@heyhaleema, I feel like this should be in another PR as this is stuff i didn't personally add for this PR. I can add this in another issue and open a PR for this

[heyhaleema]

@Lucrezia-Cester I think, ideally, every PR needs to pass our checks, regardless of whether its related to the PR-specific changes. In this case, I think if we can fix this in this PR, then it would ensure that anyone branching off main would have these checks passing from the start.

Happy to contribute to this branch and do some tidying up, if that works? I'll also update this branch with merge commit to bring in changes from main, if that's okay, too.

[Lucrezia-Cester]

@heyhaleema ah yes, sorry, good point

[heyhaleema]

@laurencejackson FYI, I've reviewed and approved @Lucrezia-Cester's changes, but do we want to also update the files with check annotations?

[laurencejackson]

@laurencejackson FYI, I've reviewed and approved @Lucrezia-Cester's changes, but do we want to also update the files with check annotations?

If you mean the warnings raised by flake8 then its not urgent. It's useful for us to fix these as we go along to ensure some code standards but I think we can go ahead and merge this once the tests have passed.