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

[python/bids_import.py] Typo in code preventing run #1108

Closed
jeffersoncasimir opened this issue Apr 19, 2024 · 0 comments · Fixed by #1129
Closed

[python/bids_import.py] Typo in code preventing run #1108

jeffersoncasimir opened this issue Apr 19, 2024 · 0 comments · Fixed by #1129
Assignees
@jeffersoncasimir
Labels
26.0-bug A-BIDS Area: BIDS. Issues and pull requests related to BIDS and the BIDS import pipeline A-EEG Area: EEG. Issues and pull requests related to EEG / iEEG processing

Comments

@jeffersoncasimir
Copy link
Contributor

jeffersoncasimir commented Apr 19, 2024 •
edited
Loading

There is a missing comma on line 357.

Found while testing #1078. This will not prevent me from otherwise testing it properly

Edit: Should also include rename of parameter below type -> dataset_type
@jeffersoncasimir jeffersoncasimir added A-BIDS Area: BIDS. Issues and pull requests related to BIDS and the BIDS import pipeline A-EEG Area: EEG. Issues and pull requests related to EEG / iEEG processing 26.0-bug labels Apr 19, 2024
@cmadjar cmadjar mentioned this issue Apr 19, 2024
Closed
5 tasks
@jeffersoncasimir jeffersoncasimir mentioned this issue May 30, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jeffersoncasimir jeffersoncasimir

Labels
26.0-bug A-BIDS Area: BIDS. Issues and pull requests related to BIDS and the BIDS import pipeline A-EEG Area: EEG. Issues and pull requests related to EEG / iEEG processing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[bids_imports pipeline] Fixed typos preventing run jeffersoncasimir/Loris-MRI
1 participant
@jeffersoncasimir