-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add set_data_io to Data #1013
Add set_data_io to Data #1013
Conversation
Thanks for working on this, @CodyCBakerPhD. I think this feature is really important. After this is integrated, I would like to also add a convenience method on |
# Conflicts: # src/hdmf/container.py
I'll take a stab and finishing this |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1013 +/- ##
=======================================
Coverage 88.57% 88.57%
=======================================
Files 45 45
Lines 9503 9509 +6
Branches 2703 2704 +1
=======================================
+ Hits 8417 8423 +6
Misses 764 764
Partials 322 322 ☔ View full report in Codecov by Sentry. |
add test for Data.set_data_io
fix #1012
Motivation
Outlined in issue
How to test the behavior?
Checklist
CHANGELOG.md
with your changes?