Remove units from the ref files

[WilliamJamieson]

Resolves RAD-192

Closes #489

This PR removes the units from the reference files.

Tasks

• Update or add relevant rad tests.
• Update relevant docstrings and / or docs/ page.
• Does this PR change any schema files?
  • Schema changes were discussed at RAD Review Board meeting.
• Does this PR change any API used downstream? (If not, label with no-changelog-entry-needed.)
  • Write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see below for change types).
  • Start a romancal regression test (https://github.com/spacetelescope/RegressionTests/actions/workflows/romancal.yml) with this branch installed ("git+https://github.com/<fork>/rad@<branch>").
  • Update relevant roman_datamodels utilities and tests.

News fragment change types:

• changes/<PR#>.feature.rst: new feature
• changes/<PR#>.bugfix.rst: fixes an issue
• changes/<PR#>.doc.rst: documentation change
• changes/<PR#>.removal.rst: deprecation or removal of public API
• changes/<PR#>.misc.rst: infrastructure or miscellaneous change

[codecov]

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 96.28%. Comparing base (d96b8da) to head (2f4033c).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
tests/test_schemas.py	85.71%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #490      +/-   ##
==========================================
- Coverage   96.59%   96.28%   -0.32%     
==========================================
  Files           4        4              
  Lines         235      242       +7     
==========================================
+ Hits          227      233       +6     
- Misses          8        9       +1     

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

[schlafly]

Thanks. This looks good. We're currently storing the units as an optional, unused bit of the schema like this:
https://github.com/spacetelescope/rad/blob/main/src/rad/resources/schemas/wfi_image-1.0.0.yaml#L105
so at least the information is in principle present in a machine readable way. Would you mind adding those bits?

I think that doesn't actually affect RTB's ability to build the files, though, so we could advertise these branches to RTB and they could develop the needed files?

[PaulHuwe]

This will also address #445 when implemented

[WilliamJamieson]

RDM failure will be fixed by spacetelescope/roman_datamodels#408.

[PaulHuwe]

LGTM

[nden]

@PaulHuwe Was there a decision to remove input_units`` and output_units`? I think especially the output units can be important.

[PaulHuwe]

You are right that these two should remain.