openmrslab / suspect

MRS processing tools
https://suspect.readthedocs.io/en/latest/
MIT License
33 stars 23 forks source link

Fix CI test and adapt to new version of dependencies #170

Closed darrencl closed 4 months ago

darrencl commented 5 months ago

Fixing CI:

Closes #162, #167

darrencl commented 5 months ago

Hi @bennyrowland ,

Here is the PR that fixed the initial CI and adapts the code to work in newer versions of dependencies. My goal here is to check that everything works as before. I would make a separate PR If there is any refactor to tidy up the code related to this changes, so that the master branch could benefit on this changes ASAP.

Thanks!

bennyrowland commented 4 months ago

@darrencl, thanks for this, it all looks fine to me - I just had one question that is more for my own understanding than anything, I am happy for you to merge it in.