set_data_io method and associated test

[bendichter]

Motivation

A potential replacement for NeurodataWithoutBorders/pynwb#1752

How to test the behavior?

Show how to reproduce the new behavior (can be a bug fix or a new feature)

Checklist

• Did you update CHANGELOG.md with your changes?
• Have you checked our Contributing document?
• Have you ensured the PR clearly describes the problem and the solution?
• Is your contribution compliant with our coding style? This can be checked running ruff from the source directory.
• Have you checked to ensure that there aren't other open Pull Requests for the same change?
• Have you included the relevant issue number using "Fix #XXX" notation where XXX is the issue number? By including "Fix #XXX" you allow GitHub to close issue #XXX when the PR is merged.

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (92915c2) 88.39% compared to head (dc2d8f5) 88.40%.

Additional details and impacted files

@@           Coverage Diff           @@
##              dev     #934   +/-   ##
=======================================
  Coverage   88.39%   88.40%           
=======================================
  Files          45       45           
  Lines        9350     9355    +5     
  Branches     2662     2663    +1     
=======================================
+ Hits         8265     8270    +5     
  Misses        766      766           
  Partials      319      319           

Files Changed	Coverage Δ
src/hdmf/container.py	91.65% <100.00%> (+0.04%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rly]

I think this is fine. The only minor concern is that this will not raise an error or do anything if a user calls set_data_io on a non-scalar attribute. But that is rare and has no bad side effect. There is no straightforward way to detect at the container level whether a field is a dataset or an attribute. I think this is good then. Please add a changelog. Thanks!

[bendichter]

@rly I just noticed there is a (now defuct) method on Data called set_dataio (no second underscore). Do you think it would be better to rename this to be consistent?

[rly]

I kind of prefer set_data_io because the underscores separate words. wrap_data_io might be better, but either is fine.