spacetelescope / romancal

Python library to process science observations from the Nancy Grace Roman Space Telescope
https://roman-pipeline.readthedocs.io/en/latest/
Other
32 stars 28 forks source link

Use Numpy arrays in data products instead of quantities #1374

Open stscijgbot-rstdms opened 2 months ago

stscijgbot-rstdms commented 2 months ago

Issue RCAL-897 was created on JIRA by Nadia Dencheva:

Currently data arrays (excluding dq) are astropy quantities, i.e. they combine arrays with astropy units

Proposal:

Pros of using astropy quantities

Cons of using astropy quantities

Timing: If accepted this change should be completed soon after Build 15 delivery on Sep 15.

Decision: This change should be agreed on by INS and SCSB

schlafly commented 2 months ago

As a note, we use quantities in a few different places:

I'm not sure I've understood yet what the separate section about units means.

It might be nice if this were in the schema around here https://github.com/spacetelescope/rad/blob/main/src/rad/resources/schemas/photometry-1.0.0.yaml#L9-L25 but instead we just had unit: "MJy.sr**-1" parallel with the description instead of the current scheme. I don't know if any of that information propagates to roman_datamodels, though, or how we would want to expose it there.

braingram commented 2 months ago

xref: https://github.com/spacetelescope/rad/pull/438