Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration with BIDScoin #199

Open
marcelzwiers opened this issue Apr 2, 2020 · 2 comments
Open

Integration with BIDScoin #199

marcelzwiers opened this issue Apr 2, 2020 · 2 comments
Labels
Enhancement New feature or request Help wanted Extra attention is needed Outreach Issue about outreaching of any form

Comments

@marcelzwiers
Copy link
Contributor

marcelzwiers commented Apr 2, 2020

I am the author of BIDScoin, a DICOM to BIDS converter:
https://github.com/Donders-Institute/bidscoin

Goal

BIDScoin doesn't convert physiological data but supports plugins and I was wondering how feasible it would be to write a plugin-wrapper around phys2bids tool(s) so that it integrates directly with BIDScoin.

How would that work and what would be the benefit?

BIDScoin would pass the heuristics (e.g. BIDS-labels) it extracted from the imaging source data to phys2bids, which would otherwise not be available so easily. It would allow to convert both the imaging and physiological data in one pass, making the BIDS output data more consistent. All this is a bit of a long shot from my side, since I don't know much about the working and design of phys2bids, I'm just exploring to see if there is common ground here and I would be very interested to hear your thoughts on this...

Cheers,
Marcel

@smoia
Copy link
Member

smoia commented Apr 2, 2020

Hello @marcelzwiers , thank you for opening this issue!
I think it could be feasible to write a plugin-wrapper around phys2bids to integrate the two programs together. Remember, however, that for the moment phys2bids does not support physiological file recorded with the MRI, but only with external instruments such as Biopac and ADInstruments. This doesn't mean we won't support them in the future, but for the moment other packages that do so exist already.

This said, the main challenge I foresee would be associating the right DICOM files to the right physiological recordings. That done, passing the right labels to the main function of phy2bids should be quite easy - especially if we refactor our code as planned in #44 to have heuristic files that are a list of if ..elif statements.

We have a developer call next Thursday, as tracked in #194. Could you/would you join? We could talk about it all together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Help wanted Extra attention is needed Outreach Issue about outreaching of any form
Projects
None yet
Development

No branches or pull requests

2 participants