GeoscienceAustralia / ga_sar_workflow

InSAR processing workflow used by Geoscience Australia
Apache License 2.0
3 stars 0 forks source link

Some more unit tests for our production scripts & dead code removal #342

Closed mwheeler closed 2 years ago

mwheeler commented 2 years ago

This PR ultimately brings us up to 80% code coverage which was our general "we want this to be the minimum we maintain" quality wise - thus addressing #277

Changes are basically:

  1. Added tests for the NCI job reporting script
  2. Fixed the job reporting script (was made for the 0.9 release when we processed eastern australia stacks)
  3. Added a basic smoke test for the NCI job submission script
  4. Removed a lot of dead code (unused code related to the old grid definition)

This also includes a little more US->AU spelling around polarisation.

mwheeler commented 2 years ago

Comments addressed in 7e5d64e