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

Bugfix megreg #619

Merged
merged 4 commits into from May 17, 2023
Merged

Bugfix megreg #619

merged 4 commits into from May 17, 2023

Conversation

rcassani
Copy link
Collaborator

This PR originated from the forum question: https://neuroimage.usc.edu/forums/t//40145 and offline discussions with @Moo-Marc. These are the changes:

  • Do not allow to have a channel shared for raw files, as this action will delete all the previous projectors from those raw files

  • The Projector field is removed from the average channel structure that is saved only using the options "shared channel" and "non-raw" files.


% Do not allow shared channel for raw data files
if isShareChan && any(strcmpi({sInputs.FileType}, 'raw'))
bst_report('Error', sProcess, [], 'Raw files should not use shared channel as all projectors from pre-processing will be lost');
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

more importantly: "as the interpolation for raw files is saved as a projector in individual channel files"

@rcassani rcassani merged commit 509d719 into brainstorm-tools:master May 17, 2023
@rcassani rcassani deleted the bugfix-megreg branch May 26, 2023 15:04
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.

None yet

2 participants