Updates for pynwb 3.1.0

[samuelbray32]

Breaking change introduced by pynwb=3.1.0

• Device objects must receive a DeviceModel object rather than a str for the model argument
• Example DeviceModel declaration:

from pynwb.device import Device, DeviceModel
DeviceModel(
    name="MyDevice",
    description="A test device",
    manufacturer="MyCompany"
    )

• Affected Locations
  • add_cameras
  • make_optogenetic_source (pending Implement addition of optogenetic metadata #118)
  • make_optical_fiber (pending Implement addition of optogenetic metadata #118)

[samuelbray32]

Need to further consider how this will work for ExcitationSource since that takes ExcitationSourceModel as the model rather than a string, so not directly compatible. Potentially ExcitationSourceModel could become a subclass of DeviceModel

Switching PR to draft pending pynwb solution to general problem

[rly]

While I work on this solution in pynwb this week, if this issue needs to be resolved ASAP for conversions, I would recommend pinning to pynwb 3.0.0.

[samuelbray32]

Thanks @rly . The main branch is already pinned to that version from a PR early this week. I'll make a PR to resolve this repo when ready, but no rush on our end

[rly]

Ah, sorry I missed that!

[samuelbray32]

Updating: Relevant PR in pynwb. Will test and see if we can unpin