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

[Bug]: add_data_interface fails silently on pynwb 3.0 #2050

Open
3 tasks done
h-mayorquin opened this issue Mar 4, 2025 · 0 comments · May be fixed by #2052
Open
3 tasks done

[Bug]: add_data_interface fails silently on pynwb 3.0 #2050

h-mayorquin opened this issue Mar 4, 2025 · 0 comments · May be fixed by #2052
Labels
category: bug errors in the code or code behavior priority: medium non-critical problem and/or affecting only a small set of NWB users

Comments

@h-mayorquin
Copy link
Contributor

h-mayorquin commented Mar 4, 2025

What happened?

It seems that now in pynwb 3.0 the method add_data_interface was replaced by add.
But the method add_data_interface does not add the structure or throws an error.

Steps to Reproduce

from pynwb.testing.mock.file import mock_NWBFile
from pynwb.testing.mock.ecephys import mock_ElectricalSeries


nwbfile = mock_NWBFile()

# Create processing modules
ecephys_module = nwbfile.create_processing_module('ecephys', 'Example ecephys data')

electrical_series = mock_ElectricalSeries()
ecephys_module.add_data_interface(electrical_series)

assert len(nwbfile.processing["ecephys"].data_interfaces) == 1
@stephprince stephprince added category: bug errors in the code or code behavior priority: medium non-critical problem and/or affecting only a small set of NWB users labels Mar 4, 2025
@stephprince stephprince linked a pull request Mar 4, 2025 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: bug errors in the code or code behavior priority: medium non-critical problem and/or affecting only a small set of NWB users
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants