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

FIXED func mask overestimates, REMOVED filtering for multi-echo #114

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

shyu2kko
Copy link

  • FIXED mask overestimates w/ tight mask on echo 1 by mri_synthstrip
  • after testing high pass filtering not significantly improving scans
  • filtering not necessary after tedana (Kundu et al., 2017 NeuroImage)

@prioux
Copy link

prioux commented Sep 11, 2024

I cannot comment on the scientific content of this PR, but as for the changes themselves, this PR add three junk MACOS-specific resource forks (see https://en.wikipedia.org/wiki/Resource_fork). These are recognizable because their basenames all start with period-underscore (._) . These files should NOT be added to the project.

  1. ._.DS_Store
  2. ._micapipe
  3. functions/._init.sh

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

Successfully merging this pull request may close these issues.

2 participants