312 bug fix update geostationary readers to support multiple scan times

[srikanth-kumar]

passes_seviri.WV-Upper.unprojected_image.log
passes_abi.static.Infrared.imagery_annotated.log
passes_ahi.tc.WV.geotiff.log

The test scripts in this branch were failing by default before my code addition I took @mindyls advice and did merge the v1.12.0-release branch into this. The three files attached above are the results of successful tests for:

1. $GEOIPS_PACKAGES_DIR/geoips/tests/scripts/abi.static.Infrared.imagery_annotated.sh
2. $GEOIPS_PACKAGES_DIR/geoips/tests/scripts/ahi.tc.WV.geotiff.sh
3. $GEOIPS_PACKAGES_DIR/geoips/tests/scripts/seviri.WV-Upper.unprojected_image.sh

Reviewer Checklist

PR author: Please ensure you meet all of the below requirements, and check boxes appropriately.

Reviewers: Please confirm all required testing/documentation has been completed prior to approving.

Remove lines that are not applicable, explain if you select "NO REQUIRED"

• Required existing tests pass (ie full_test.sh, others as appropriate)
• NO REQUIRED existing tests (explain why not required)
• Required unit tests added and pass for new/modified functionality
• NO REQUIRED unit tests (explain why not required)
• Required integration tests added and pass for new/modified functionality
• NO REQUIRED integration tests (explain why not required)
• Required documentation added for new/modified functionality
• NO REQUIRED documentation (explain why not required)
• Required release notes added for new/modified functionality
• NO REQUIRED release notes (explain why not required)
• Required updates to other repos complete
• NO REQUIRED updates to other repos (explain why not required)

https://github.com/NRLMMD-GEOIPS/.github/blob/main/.github/review-template.md

Related Issues

fixes NRLMMD-GEOIPS/geoips#NNN

Testing Instructions

Summary

Output

[evrose54]

See issue #802.

[evrose54]

See issue #803.

[evrose54]

See issue #804.

[evrose54]

After running full test, these checks fail for me with the new changes. We should likely update the expected output of these tests to include our new changes. Technically, the source_file_names attribute is wrong, as it references all test datasets initially sent to the reader, but I've tested trying to only gather the needed file names and this is throwing a wrench in things and is very complex. We might want to talk about that more. Nonetheless, all other tests pass, so this is more of a concern about metadata rather than correct functionality.

Failing tests:

abi.config_based_output_low_memory.sh
abi.config_based_output.sh
abi.tc.Visible.imagery_clean.sh

ami.tc.WV.geotiff.sh is also failing for me but that is not a cause of this PR. This has been happening for a long time even on main for me.