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

Added FAQ: ''why should I set cfg.continuous = 'yes' when preprocessing CTF trial-based data?'' #806

Merged
merged 5 commits into from
Sep 20, 2024

Conversation

contsili
Copy link
Contributor

As discussed with @schoffelen, this question was raised by @contsili and other external FieldTrip users.

Perhaps a hyperlink from the preprocessing tutorial to the FAQ could clarify why cfg.continuous is set to 'yes' in the tutorial. Many users copy the tutorial to their own analysis and use cfg.continuous='yes' for EEG or MEG data (other than CTF) without understanding the reasoning behind it.

Small note: I could not clone two files. I believe this is a windows problem (see commit be45c95).

script .md and reproducescript .md have a space between the name of the file and the file extension. Thus I cannot clone them in my local windows machine.
@robertoostenveld
Copy link
Member

HI @contsili, the "tag" files are auto-generated/updated whenever the website content is changed. That applies both to the _data/tag/xxx.yml files and the tag/xxx.md files. The two that you (accidentally) deleted are weird as they both seem to have a space in the filename. I suspect they were created in error. Filenames should be simple, hence I will also rename your page...

@robertoostenveld
Copy link
Member

You should also add the page to the preprpocessing section in the faq.md file.

@contsili
Copy link
Contributor Author

I will do it.

@contsili
Copy link
Contributor Author

contsili commented Sep 19, 2024

Now it should be ready to merge.

@robertoostenveld robertoostenveld merged commit 9715cc4 into fieldtrip:master Sep 20, 2024
schoffelen pushed a commit to schoffelen/website that referenced this pull request Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants