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 verbosity args to windowers #391

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

gemeinl
Copy link
Collaborator

@gemeinl gemeinl commented Jun 22, 2022

No description provided.

@codecov
Copy link

codecov bot commented Jun 22, 2022

Codecov Report

Merging #391 (cbbf33e) into master (712f66d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #391   +/-   ##
=======================================
  Coverage   82.66%   82.66%           
=======================================
  Files          53       53           
  Lines        3738     3738           
=======================================
  Hits         3090     3090           
  Misses        648      648           

@agramfort
Copy link
Collaborator

running something like this mne.set_config("MNE_LOGGING_LEVEL", "WARNING") should otherwise silence mne verbosity

@gemeinl
Copy link
Collaborator Author

gemeinl commented Jul 22, 2022

running something like this mne.set_config("MNE_LOGGING_LEVEL", "WARNING") should otherwise silence mne verbosity

Unfortunately, for me, this does not seem to work. Also mne.set_log_level('ERROR') which I used before does not work.
I suspect that parallelization through joblib resets / removes / does not inherit the logging level?
My output is flooded while indexing individual compute windows with Loading data for 1 events and 6000 original time points ...

@bruAristimunha
Copy link
Collaborator

@gemeinl, It doesn't work for me either, but only when I process in parallel. Is it your case?

@bruAristimunha
Copy link
Collaborator

@agramfort, can you merge this option?

@agramfort agramfort merged commit a5e52d5 into braindecode:master Aug 24, 2022
bruAristimunha added a commit to bruAristimunha/braindecode that referenced this pull request Sep 12, 2022
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.

3 participants