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

Don't insert superfluous newlines in subprocess log messages #11219

Merged
merged 3 commits into from
Oct 3, 2022

Conversation

hoechenberger
Copy link
Member

@hoechenberger hoechenberger commented Oct 1, 2022

Fixes #11218

@larsoner could you take a look please? This seems to resolve my issue.

I believe the logger automatically appends a newline character, so we end up with two subsequent ones at the end of a line if the input string already contains a \n at the end.

@hoechenberger hoechenberger marked this pull request as ready for review October 3, 2022 10:48
@larsoner larsoner merged commit db0d02b into mne-tools:main Oct 3, 2022
@larsoner
Copy link
Member

larsoner commented Oct 3, 2022

Thanks @hoechenberger

@hoechenberger hoechenberger deleted the hoechenberger/issue11218 branch October 3, 2022 15:39
larsoner added a commit to drammock/mne-python that referenced this pull request Oct 4, 2022
* upstream/main:
  Don't insert superfluous newlines in subprocess log messages (mne-tools#11219)
  purge _get_args helper func (mne-tools#11215)
larsoner added a commit to sappelhoff/mne-python that referenced this pull request Oct 4, 2022
* upstream/main:
  Don't insert superfluous newlines in subprocess log messages (mne-tools#11219)
  purge _get_args helper func (mne-tools#11215)
  Standardize topomap args (mne-tools#11123)
  MAINT: Ensure no datasets are downloaded in tests (mne-tools#11213)
larsoner added a commit to larsoner/mne-python that referenced this pull request Oct 11, 2022
* upstream/main: (64 commits)
  MAINT: Better check (mne-tools#11229)
  MAINT: Fix link and update instantiation note (mne-tools#11228)
  BUG: Add estimated fiducials when missing / assumed head coords (mne-tools#11212)
  Fix tfr db (mne-tools#11223)
  MAINT: Update link (mne-tools#11222)
  add CPGRL doc section (mne-tools#11216)
  Don't insert superfluous newlines in subprocess log messages (mne-tools#11219)
  purge _get_args helper func (mne-tools#11215)
  Standardize topomap args (mne-tools#11123)
  MAINT: Ensure no datasets are downloaded in tests (mne-tools#11213)
  MAINT: Fix Cirrus caching (mne-tools#11211)
  Fix mesh display in tutorial (mne-tools#11200)
  MAINT: Add arm64 CI using CirrusCI (mne-tools#11209)
  Fix spatial colors (mne-tools#11201)
  MAINT: Fix CircleCI error (mne-tools#11205) [circle deploy]
  Add regression-based approach to removing EOG artifacts (mne-tools#11046)
  [DOC, MRG] Minor documentation improvements and remove glossary entry for array-like (mne-tools#11207)
  Fix `include_tmax` not considered in `mne.io.Raw.crop` to check `tmax` in bounds (mne-tools#11204)
  MAINT: Fix notebook backend (mne-tools#11206)
  MRG: Fix displayed Raw duration in Jupyter notebook (mne-tools#11203)
  ...
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.

mne.utils.run_subprocess() inserts undesired empty lines in stdout
2 participants