nipype icon indicating copy to clipboard operation
nipype copied to clipboard

Please consider making nipype compatible with traits >= 6.4

Open penguinpee opened this issue 1 year ago • 1 comments
trafficstars

I'm aware that traits 6.4.0 introduced breaking changes and that for that reason the version is pinned to <6.4 (#3501). I'm also aware that there are currently no plans to overhaul nipype.

However, something will break eventually, e.g with a newer Python release, requiring adaption to a newer traits version. Since current supported Fedora releases have all updated traits to >6.4, it's preventing me from packaging nipype for Fedora.

I'm not sure it's within my capabilities to help refactoring (probably not). But is there some documentation, ongoing work or similar, which would allow me to get an idea what needs changing where and what possible culprits may be?

So far, I've seen a single test out of the 2000+ tests failing due to the newer traits version. That's running tests without optional dependencies, which may hide a few additional failures.

penguinpee avatar Jul 25 '24 10:07 penguinpee

same pain on Debian

a-detiste avatar Aug 30 '24 19:08 a-detiste

Sorry, I thought I had responded to this, but it was probably in some tab that got closed.

When traits 6.4 came out, we saw many more than 1 test fail, so it's possible that they've re-added compatibility shims that would make the process less daunting. If you have a PR to resolve the issue, I'd be willing to review it and get it in.

One caution: Nipype has severe technical debt, most notably in the CI infrastructure. There is very little appetite to spend time on this project, not least because getting it back up to being properly tested would be a massive time sink. I'd be willing to onboard new maintainers, if you've got the interest and time.

effigies avatar Aug 30 '24 20:08 effigies