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

Update to output neuro/conmat datatype. #5

Open
soichih opened this issue Sep 13, 2019 · 2 comments
Open

Update to output neuro/conmat datatype. #5

soichih opened this issue Sep 13, 2019 · 2 comments

Comments

@soichih
Copy link
Contributor

soichih commented Sep 13, 2019

Is it possible to update this App to output the neuro/conmat datatype like "fMRI to Connectivity Matrices" App? (https://brainlife.io/app/5c720cf63e2f2c0030a23486)

Here is the datatype description

https://brainlife.io/datatypes/5d34d9f744947d8aea0e0d2f

cm.csv and label.json should look like this.

Screenshot from 2019-09-13 14-00-02
Screenshot from 2019-09-13 13-59-53

@bcmcpher
Copy link
Owner

The .csv is easy enough to set up - it writes that now.

But how am I supposed to handle multiple / optional outputs? Currently this can make up to 2 different edge weights (density, average property), but in theory it can scale to many more than this (count, length, density corrected for length, multiple tensor/noddi values). Can these just be tagged appropriately? Can we even work with optional outputs yet?

And there's nothing I can do about label.json in this app. This has to come with the parcellation that is passed. There is no way for me to figure out what the labels are supposed to be when I only get parc.nii.gz.

@soichih
Copy link
Contributor Author

soichih commented Sep 13, 2019

I believe other outputs can be added as optional output for neuro/conmat (right now there is only cm.csv that's listed). If there are standard measures that should be required across other connectivity Apps, we can make them required as well. Right now this datatype is used by fMRI to connectivity matrices app. I will ping @faskowit about this.

You can convert the key.txt that's part of parcellation/volume input to label.json. Like this

https://github.com/faskowit/app-fmri-2-mat/blob/0.0.4/generate_cm_datatype.py#L12

Please note.. as part of our datatype discussion about parcellation/volume, I am planning to propose parcellation/volume datatype to use label.json format instead of key.txt. When this happens, you should just have to copy the label.json into neuro/conmat datatype instead of parsing key.txt.. I will push for that soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants