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

[BUG] c02_save_separate_run.py #49

Open
Michael-Sun opened this issue Oct 26, 2023 · 1 comment
Open

[BUG] c02_save_separate_run.py #49

Michael-Sun opened this issue Oct 26, 2023 · 1 comment
Labels
modules: physio Physiological data

Comments

@Michael-Sun
Copy link

Which module is this from?

physio

What is the issue?

No warning or error message is displayed when physio file is processed by there is insufficient metadata to account for all of the runs detected.

What was your expected behavior?

File will be processed and all runs will be output, but named according to the available metadata. A warning or error should be thrown to alert the user that this has happened.

How can we reproduce this?

run c02_save_separate_run.py on an acqknowledge file with more runs than metadata has names for.

Any additional context?

(Write your answer here.)

@Michael-Sun
Copy link
Author

Turns out this issue also occurs when the metadata specifies more than the number of runs that the script detects in the acqknowledge file.

@jungheejung jungheejung added the modules: physio Physiological data label Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
modules: physio Physiological data
Projects
None yet
Development

No branches or pull requests

2 participants