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

Add fmrwhy to multi-echo resources #683

Closed
emdupre opened this issue Feb 17, 2021 · 4 comments · Fixed by #948
Closed

Add fmrwhy to multi-echo resources #683

emdupre opened this issue Feb 17, 2021 · 4 comments · Fixed by #948
Labels
documentation issues related to improving documentation for the project good first issue issues that we think are good for new contributors. Equivalent to "very low" effort.

Comments

@emdupre
Copy link
Member

emdupre commented Feb 17, 2021

Summary

Recently, @jsheunis introduced fmrwhy (https://fmrwhy.readthedocs.io/), a MATLAB package for processing multi-echo data. It would be great to include this in our documentation as a resource for the multi-echo community !

Additional Detail

I think it would need to be added here under Multi-echo processing software, but pending #677 the exact text area could be updated.

Next Steps

Open a PR to add this in 😸

@tsalo tsalo added documentation issues related to improving documentation for the project good first issue issues that we think are good for new contributors. Equivalent to "very low" effort. labels Feb 17, 2021
@zswang-brainimaging
Copy link

@emdupre Thank you for this information! Will see if something can be combined with your toolbox.

@emdupre
Copy link
Member Author

emdupre commented Feb 17, 2021

@emdupre Thank you for this information! Will see if something can be combined with your toolbox.

That's great, @zswang-brainimaging ! Glad to hear that it's useful to you. If you're primarily working in MATLAB, fmrwhy seems like an especially important resource.

It's always challenging to integrate MATLAB and Python environments, but if you find a path using both fmrwhy and tedana that's useful for you, definitely let us know !

@handwerkerd
Copy link
Member

I'm trying to go through old issues to see what we can address. I can definitely add fmrwhy to the docs, but it looks like it hasn't had any commits in over a year. Since we regularly get questions on using multi-echo data with SPM it's very worth adding, but I want an accurate description.
@jsheunis, Should the descriptive text me something like:
"fmrwhy runs BIDS-compatible fMRI analysis with SPM12 and supports multi-echo data. It is no longer being actively maintained"

@jsheunis
Copy link
Contributor

@handwerkerd thanks for the message, that is accurate yes. I'll update the repo with a similar note.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation issues related to improving documentation for the project good first issue issues that we think are good for new contributors. Equivalent to "very low" effort.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants