search

ME-ICA / aroma

ICA-AROMA, as a Python package. A work in progress.
Apache License 2.0
7 stars 11 forks source link

Enforce BIDS Derivatives-compatible ICA inputs #17

Open tsalo opened 4 years ago

tsalo commented 4 years ago

We should restructure the ICA outputs that ICA-AROMA ingests to be BIDS Derivatives-compatible. In order to maintain support for MELODIC, I think we need a new function that converts MELODIC outputs to a BIDS Derivatives outputs. The main ICA-AROMA functions would then just use BIDS Derivatives outputs.

oesteban commented 3 years ago

Totally in favor of this.

I think we need a new function that converts MELODIC outputs to a BIDS Derivatives outputs. The main ICA-AROMA functions would then just use BIDS Derivatives outputs.

Yes, but let's not make that function part of this issue and give it a super-low priority.

oesteban commented 3 years ago

In other words, I would replace "Accept" with "Enforce" in the title of this issue, and assume that the input components are BIDS compliant.

tsalo commented 3 years ago

That works for me. I think a MELODIC-to-BIDS Derivatives function should go in a separate repository anyway. I would just prefer to have it as a Python function if at all possible, so that AROMA users won't need to get nipype working to use it.

oesteban commented 3 years ago

I would just prefer to have it as a Python function if at all possible

Makes sense. If this project progresses quickly, that could be a nice feature to have 👍