Skip to content

hotfix#605

Merged
gviejo merged 1 commit into
devfrom
fix-xml
May 26, 2026
Merged

hotfix#605
gviejo merged 1 commit into
devfrom
fix-xml

Conversation

@gviejo

@gviejo gviejo commented May 21, 2026

Copy link
Copy Markdown
Contributor

This pull request makes a small but important fix to the initialization logic in pynapple/io/interface_neurosuite.py. The update ensures that the skip and groups dictionaries are correctly keyed by channel ID, improving data consistency and preventing potential indexing errors.

  • Bug fix:
    • Updated assignment to self.skip and self.groups to use ch_id as the key instead of the loop counter count, ensuring correct mapping of channel metadata.

@codecov

codecov Bot commented May 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
pynapple/io/interface_neurosuite.py 93.15% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gviejo gviejo merged commit 74ed87d into dev May 26, 2026
29 of 31 checks passed
@gviejo gviejo deleted the fix-xml branch May 26, 2026 16:35
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.

1 participant