search

DES-SL / Y6_Bulk_Coadd_Cutouts

Codebase for large-scale cutout production from DES co-added images on the FNAL cluster
MIT License
0 stars 2 forks source link

Standardizing CUTOUT_ID Contents #23

Open kadrlica opened 3 years ago

kadrlica commented 3 years ago

@Rob Morgan I noticed an inconsistency between the DES and DELVE cutout file structure. Both have a TableHDU named CUTOUT_ID, but the single column in this table for DES is named COADD_OBJECT_ID while in DELVE it is CUTOUT_ID. To allow for more flexibility, I suggest standardizing it to ID or something like that.

rmorgan10 commented 3 years ago

Sounds good that's something I can implement

rmorgan10 commented 3 years ago

let's just go with ID for both the HDU and the column name

rob commented 3 years ago

Not sure I’ll take a look later!

On Fri, Feb 12, 2021 at 16:42 Alex Drlica-Wagner notifications@github.com wrote:

@rob https://github.com/rob Morgan I noticed an inconsistency between the DES and DELVE cutout file structure. Both have a TableHDU named CUTOUT_ID, but the single column in this table for DES is named COADD_OBJECT_ID while in DELVE it is CUTOUT_ID. To allow for more flexibility, I suggest standardizing it to ID or something like that.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/DES-SL/Y6_Bulk_Coadd_Cutouts/issues/23, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAAD57ECCD2G7QLT46ADPTS6WOCXANCNFSM4XRMCWTQ .