This stems from this comment from PR #530 in which I had a failing test on abi.static.Infrared.netcdf_geoips.sh. This failed because the compare product had not been updated to reflect the new Denver sector plugin. When showing this to @jsolbrig, his tests were passing and mine weren't. This is due to the fact that his output product was not being overwritten by the output formatters. We should look into netcdf output formatters and ensure that even if a netcdf file already exists, it should be properly overwritten that the previous file persisting.
Code to demonstrate issue
abi.static.Infrared.netcdf_geoips.sh did this. It might not be a consistent error for everyone for the reasons described above.
Checklist for Completion
[ ] Address all netcdf-based output formatters and make sure they properly overwrite.
Requested Update
Description
This stems from this comment from PR #530 in which I had a failing test on
abi.static.Infrared.netcdf_geoips.sh. This failed because the compare product had not been updated to reflect the new Denver sector plugin. When showing this to @jsolbrig, his tests were passing and mine weren't. This is due to the fact that his output product was not being overwritten by the output formatters. We should look into netcdf output formatters and ensure that even if a netcdf file already exists, it should be properly overwritten that the previous file persisting.Code to demonstrate issue
abi.static.Infrared.netcdf_geoips.shdid this. It might not be a consistent error for everyone for the reasons described above.Checklist for Completion