Add grid_spacing and origin_coords to the fields of ImagingPlane

[h-mayorquin]

Motivation

This should fix #1885

I discussed this with @rly and it seems that it was just an omission. This PR tries to address this. So far, this seems to work for grid_spacing but for some reason that escapes me it seems that origin_coords is not added to the fields.

from pynwb.testing.mock.ophys import mock_ImagingPlane

imaging_plane = mock_ImagingPlane(grid_spacing=[1.0, 1.0], origin_coords=[0.0, 0.0])
imaging_plane.origin_coords
list(imaging_plane.fields.keys())

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 flake8 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

All modified and coverable lines are covered by tests ✅

Project coverage is 92.01%. Comparing base (6cfcf39) to head (242cc90).

❗ Current head 242cc90 differs from pull request most recent head a5aa114. Consider uploading reports for the commit a5aa114 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##              dev    #1892   +/-   ##
=======================================
  Coverage   92.01%   92.01%           
=======================================
  Files          27       27           
  Lines        2619     2619           
  Branches      685      685           
=======================================
  Hits         2410     2410           
  Misses        139      139           
  Partials       70       70           

Flag	Coverage Δ
integration	72.81% <ø> (ø)
unit	83.92% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[rly]

So far, this seems to work for grid_spacing but for some reason that escapes me it seems that origin_coords is not added to the fields.

@h-mayorquin I think you were missing a comma in the __nwbfields__ list. I fixed that and I think it works now. Could you check, and if it looks good, add a changlog?

[h-mayorquin]

So far, this seems to work for grid_spacing but for some reason that escapes me it seems that origin_coords is not added to the fields.

@h-mayorquin I think you were missing a comma in the __nwbfields__ list. I fixed that and I think it works now. Could you check, and if it looks good, add a changlog?

My bad, I tested your change and it is working now. Thanks.

Added the changelog.

[rly]

Thanks for the fix @h-mayorquin !