Skip to content

Fix Thielen2021 paper DOI pointing to wrong publication#1055

Draft
Copilot wants to merge 2 commits intodevelopfrom
copilot/fix-thielen2021-paper-link
Draft

Fix Thielen2021 paper DOI pointing to wrong publication#1055
Copilot wants to merge 2 commits intodevelopfrom
copilot/fix-thielen2021-paper-link

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 8, 2026

The Thielen2021 dataset had its doi set to the dataset repository DOI (10.34973/9txv-z787) instead of the journal paper DOI, causing the "Read Paper" button in the API docs to resolve to the wrong publication.

Change

  • moabb/datasets/thielen2021.py: Replace the doi in Thielen2021.__init__ with the correct paper DOI:
# Before
doi="10.34973/9txv-z787",  # dataset repo DOI

# After
doi="10.1088/1741-2552/abecef",  # journal paper DOI

The dataset repository DOI is already correctly referenced via external_links and DocumentationMetadata.data_url elsewhere in the class.

Copilot AI linked an issue Apr 8, 2026 that may be closed by this pull request
Agent-Logs-Url: https://github.com/NeuroTechX/moabb/sessions/07cd95fd-5be9-4b4f-ab04-dae4a0475130

Co-authored-by: bruAristimunha <42702466+bruAristimunha@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix incorrect link for Thielen2021 dataset paper Fix Thielen2021 paper DOI pointing to wrong publication Apr 8, 2026
Copilot AI requested a review from bruAristimunha April 8, 2026 13:39
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.

[Dataset] Issue with Thielen2021

2 participants