CLEANUP - modulenames - #221
Conversation
|
Considering the lack of criticism as approval, I am going to merge this PR by the end of the week. |
|
Sounds good I think The renaming of modules will lead to lots of manual cosmetic changes to old tasklist.xml files (e.g. Cam-CAN used a bunch of roi extraction modules throughout multiple streams for its different releases) but that's not a major issue, and I support adopting a more structured naming scheme (I am assuming these module names are more consistent than before!). As for deleting modules. I didn't see these listed in Cam-CAN's latest tasklist, so that shouldn't cause a compatibility problem for us (perhaps they are old WIP modules anyway(?)): |
|
Thanks, @ethanknights, so you are saying it is good to go? |
jones-michael-s
left a comment
There was a problem hiding this comment.
Looks good to me.
|
Thank you, @jones-michael-s. I am going to merge now. |
No description provided.