Closed 36000 closed 4 months ago
Hello @36000! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
@arokem this is ready for review/merge
I think this is good to go, lets wait for #1124
@arokem i added some testing and modified the criteria system to not use pimms
@arokem this is ready for another round of review
This refactors the massive segmentation class into many files. It should make it a lot easier to change segmentation code in the future, ie, to make it so bundles can depend on the position of other bundles. It also makes recobundles into "just another" step in the pipeline, not as an entirely separate pipeline.