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

Improve documentation on masking options #898

Closed
handwerkerd opened this issue Nov 17, 2022 · 0 comments · Fixed by #948
Closed

Improve documentation on masking options #898

handwerkerd opened this issue Nov 17, 2022 · 0 comments · Fixed by #948
Labels
documentation issues related to improving documentation for the project effort: low Theoretically less than a day's work impact: medium Improves code/documentation functionality for some users priority: medium Should get addressed soon

Comments

@handwerkerd
Copy link
Member

Summary

There have been several situations were the automasking that is built into tedana spectacularly fails. We should trace this down, but our sense is that many uses are providing their own masks anyway and that avoids this problem. We should add to the documentation make clear that this is a known issue that is resolved with user-provided masks.

Additional Detail

Next Steps

  • Supplement documentation
  • If there are potential code changes to improve how masking is handed in tedana, open a new issue for that
@handwerkerd handwerkerd added documentation issues related to improving documentation for the project priority: medium Should get addressed soon effort: low Theoretically less than a day's work impact: medium Improves code/documentation functionality for some users labels Nov 17, 2022
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 effort: low Theoretically less than a day's work impact: medium Improves code/documentation functionality for some users priority: medium Should get addressed soon
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant