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.

Sign up for GitHub

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

Jump to bottom

Column names in orthogonalized mixing matrix do not match ones in metrics file #904

Closed
tsalo opened this issue Nov 22, 2022 · 0 comments · Fixed by #906
Closed

Column names in orthogonalized mixing matrix do not match ones in metrics file #904

tsalo opened this issue Nov 22, 2022 · 0 comments · Fixed by #906
Labels
bug issues describing a bug or error found in the project good first issue issues that we think are good for new contributors. Equivalent to "very low" effort.

Comments

@tsalo
Copy link
Member

tsalo commented Nov 22, 2022

Summary

The desc-ICAOrth_mixing file has column names with ICA in lower case (e.g., ica_00), but the main mixing matrix and metrics files both use upper case column names.
@tsalo tsalo added the bug issues describing a bug or error found in the project label Nov 22, 2022
@eurunuela eurunuela added the good first issue issues that we think are good for new contributors. Equivalent to "very low" effort. label Nov 22, 2022
@pablosmig pablosmig mentioned this issue Nov 23, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug issues describing a bug or error found in the project good first issue issues that we think are good for new contributors. Equivalent to "very low" effort.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[FIX] Use capital names in desc-ICAOrth_mixing.tsv columns pablosmig/tedana
2 participants
@tsalo @eurunuela