Skip to content

Pre-release 1.6.1 - #1167

Merged
bruAristimunha merged 2 commits into
NeuroTechX:developfrom
bruAristimunha:pre-release-1.6.1
Aug 27, 2026
Merged

Pre-release 1.6.1#1167
bruAristimunha merged 2 commits into
NeuroTechX:developfrom
bruAristimunha:pre-release-1.6.1

Conversation

@bruAristimunha

Copy link
Copy Markdown
Collaborator

Bug-fix release.

Version

moabb.__version__ 1.6.1dev0 → 1.6.1, propagated to CITATION.cff (+ date-released: 2026-08-27), README.md and docs/source/cite.rst. All five citation surfaces verified equal at 43 authors; CITATION.cff validates with cffconvert. The author list is unchanged — every entry in this release is by an existing author.

Changelog

Version 1.6.1 (Source - GitHub)(Stable - PyPi), and a fresh Version 1.7 (Source - GitHub) section opened.

What is in it

#1161 BNCI2025_001 label leakage — 7 kinematic channels (target position, hand velocity) were typed eeg and picked as classifier features
#1161 The Rodrigues2017 / Cattan2019_PHMD / Cattan2019_VR montages :gh:700 announced and never implemented
#1161 Cattan2019_PHMD interval described 60 s relaxation blocks as 1 s
#1161 NEMAR prefetch trusted any non-empty store, so a per-subject caller silently reached the upstream host
#1161 import moabb mutated the caller's global matplotlib rcParams
#1161 The monthly download job ran pytest download.py — a file that does not exist — and had failed every month since March
#1166 AcquisitionMetadata.n_channels derived from channel_types instead of stored beside it; 34 datasets disagreed with themselves

Closes #1162, #1163, #1165.

Bug-fix release. Bumps the version and rolls the changelog.

- moabb.__version__: 1.6.1dev0 -> 1.6.1
- CITATION.cff, README.md and docs/source/cite.rst updated to 1.6.1;
  date-released 2026-08-27. The author list is unchanged: every entry in this
  release is by an existing author.
- whats_new: stamp "Version 1.6.1 (Source - GitHub)" as "(Stable - PyPi)" and
  open a "Version 1.7 (Source - GitHub)" section.

Contents: the BNCI2025_001 label leak (seven kinematic channels typed eeg and
picked as features), the Rodrigues2017 / Cattan2019_PHMD / Cattan2019_VR
montages NeuroTechXgh-700 announced and never shipped, Cattan2019_PHMD's interval
describing 60 s blocks as 1 s, the NEMAR prefetch trusting any non-empty
store, importing moabb restyling the caller's global matplotlib rcParams, the
monthly download job pointing at a file that does not exist, and
AcquisitionMetadata.n_channels being derived rather than stored.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8d03a54acd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread moabb/__init__.py
@@ -1,5 +1,5 @@
# flake8: noqa
__version__ = "1.6.1dev0"
__version__ = "1.6.1"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Publish the breaking API change under a minor version

Releasing this code as 1.6.1 makes a patch upgrade from 1.6.0 backward-incompatible: the release notes explicitly state that AcquisitionMetadata(..., n_channels=...) is no longer accepted (docs/source/whats_new.rst:51-53), although that argument was part of the public constructor in 1.6.0. Existing consumers will therefore start raising TypeError after an ordinary patch upgrade; either retain a compatibility path for n_channels or release this as the already-opened 1.7 line.

Useful? React with 👍 / 👎.

Released sections write '- None.'; '- None yet.' is for the open section
at the top. 1.6.0 already follows this.
@bruAristimunha
bruAristimunha merged commit ad1ec68 into NeuroTechX:develop Aug 27, 2026
14 checks passed
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.

download-test.yml collects no tests, so every @pytest.mark.download test runs in no CI job

1 participant