Skip to content

Conversation

@dissagaliyeva
Copy link
Contributor

@dissagaliyeva dissagaliyeva commented Oct 16, 2022

Reference issue

Closes #11252

What does this implement/fix?

Adds the missing verbose command to the combine_channels function.

Additional information

Changed three lines of code:

  1. Added @verbose before the function call
  2. Added verbose=None to the function arguments
  3. Added %(verbose)s to the docstring

@welcome
Copy link

welcome bot commented Oct 16, 2022

Hello! 👋 Thanks for opening your first pull request here! ❤️ We will try to get back to you soon. 🚴🏽‍♂️

@agramfort
Copy link
Member

@dissagaliyeva can I ask you to add a line in latest.inc with the change and your name in names.inc?

🙏

@dissagaliyeva
Copy link
Contributor Author

@agramfort hi! is it necessary to do so if our names have already been included in this PR #11245?

@agramfort
Copy link
Member

indeed let's say no. Your name will appear in changelog then already.

@larsoner larsoner enabled auto-merge (squash) October 17, 2022 14:11
@larsoner larsoner disabled auto-merge October 17, 2022 14:12
@larsoner larsoner merged commit 4957469 into mne-tools:main Oct 17, 2022
@welcome
Copy link

welcome bot commented Oct 17, 2022

🎉 Congrats on merging your first pull request! 🥳 Looking forward to seeing more from you in the future! 💪

@larsoner
Copy link
Member

Thanks @dissagaliyeva !

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.

Missing verbose argument in mne.channels.channels.combine_channels

3 participants