populate_mosaic_basic overwritten in step

[braingram]

The metadata blending performed by populate_mosaic_basic:

romancal/romancal/resample/resample.py

Line 875 in 826a972

def populate_mosaic_basic(

is later overwritten by:

• update_exposure_times:

  romancal/romancal/resample/resample.py

  Line 533 in 826a972

  def update_exposure_times(self, output_model, exptime_tot):

• l2_into_l3_meta:

  romancal/romancal/resample/resample.py

  Line 734 in 826a972

  def l2_into_l3_meta(l3_meta, l2_meta):

The test PR #1597 adds a test that runs the resample step on 2 inputs with different metadata:

                {
                    "meta.observation.visit": 1,
                    "meta.observation.pass": 1,
                    "meta.observation.segment": 1,
                    "meta.observation.program": 1,
                    "meta.instrument.optical_element": "F158",
                    "meta.instrument.name": "WFI",
                },
                {
                    "meta.observation.visit": 2,
                    "meta.observation.pass": 2,
                    "meta.observation.segment": 2,
                    "meta.observation.program": 2,
                    "meta.instrument.optical_element": "F062",
                    "meta.instrument.name": "WFI",
                },
            ),

The mosaic model contains:

            {
                "visit": 1,
                "pass": 1,
                "segment": 1,
                "optical_element": "F158",
                "instrument": "WFI",
            },

This shows that the metadata assigned by populate_mosaic_basic is being overwritten since it would set:

• visit: -1
• pass: -1
• segment: -1

[schlafly]

Thanks. Looking at the code here---the update_exposure_times bit is redundant but at least the calculations look the same. Meanwhile l2_into_l3_meta "contributes" only coordinates and so should probably get deleted with the "coordinates" contribution folded into populate_mosaic_basic; the other bits it does are inferior to what's happening in populate_mosaic_basic.

Anyway, I agree that overwriting without the blending is wrong and the redundant calculations are problematic---just making sure we agree on the scope of the bug here. Thanks!

[nden]

@stscijgbot-rstdms

[stscijgbot-rstdms]

This issue is tracked on JIRA as RCAL-1006.