Skip to content

DOC: Add missing links to supported data format tables in events tutorialSusani/explore docs #13715

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

Open
Susaniaariya wants to merge 3 commits into
base: main
Choose a base branch
from
+3 −3
Open

DOC: Add missing links to supported data format tables in events tutorialSusani/explore docs #13715

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
522d5fa
DOC: link to import tables in events tutorial
Susaniaariya Mar 2, 2026
57e0d46
DOC: use :doc: links for MEG/EEG tutorials
Susaniaariya Mar 2, 2026
4bee2c4
Merge branch 'main' into susani/explore-docs
Susaniaariya Mar 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions tutorials/intro/20_events_from_raw.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,9 @@
# magnitudes. You can also see that every time there is a pulse on one of the
# other STIM channels, there is a corresponding pulse on ``STI 014``.
#
# .. TODO: somewhere in prev. section, link out to a table of which systems
# have STIM channels vs. which have marker files or embedded event arrays
# (once such a table has been created).
# For a list of which EEG/MEG systems use STIM channels versus marker files
# or embedded arrays, see the "Supported data formats" tables in
# :doc:`/auto_tutorials/io/10_reading_meg_data` and :doc:`/auto_tutorials/io/20_reading_eeg_data`.
Copy link
Member

@drammock drammock Mar 2, 2026

Choose a reason for hiding this comment

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

neither of the linked pages contain a table called "supported data formats".

Copy link
Author

@Susaniaariya Susaniaariya Mar 4, 2026

Choose a reason for hiding this comment

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

Hi Daniel,
Thank you for your feedback! I’ve addressed the TODO comment in 20_events_from_raw.py by adding functional cross-references to the MEG and EEG tutorials. I also clarified the surrounding comment to make the explanation more precise.
The changes are now pushed to my branch, and I’ve opened a pull request for review. Please let me know if there are any further improvements you’d like me to make.
Best regards,
Susaniaariya

#
#
# Converting a STIM channel signal to an Events array
Expand Down
Loading