Skip to content

Incremental updates needed for Eval-20 and 21

David Joy requested to merge dev/incremental-updates-eval21 into main

Merge request template for feature branches. [click to expand]

See https://semver.org for versioning information:

Given a version number MAJOR.MINOR.PATCH, increment the:

   MAJOR version when you make incompatible API changes,
   MINOR version when you add functionality in a backwards compatible manner, and
   PATCH version when you make backwards compatible bug fixes.

Major version number is 0 for no expectation of backwards compatibility.

"feature" template should be used for significant features that increment GEOWATCH's minor version number. (Ex. 0.5.0 -> 0.6.0)

"patch" template should be used for bugfixes or minor features that increment GEOWATCH's patch version number. (Ex. 0.5.0 -> 0.5.1)

Short Description

Various tweaks to support incremental mode used in Eval 20 and 21. Created a separate script for run_bas_fusion specifically for incremental mode called run_bas_fusion_incremental (includes additional code paths in support of incremental mode, didn't want to muddy up the main batch mode script).

Related MRs, Issues, and Other Links

Builds on !34 (closed)

Types of Changes

  • Documentation/tests
  • Bug fix
  • New feature
  • Breaking change

Changelog

Fixed

  • Fixed error when attempting to concatenate kwcoco videos with no images
  • Added fixup for rare issue where output site models and site summaries could be multipolygons

Added

  • Created dedicated run_bas_fusion script for incremental mode

Changed

  • Performer suffix no longer required for run_fixups
  • Changed smartflow_egress to return dictionary of assets and asset paths (rather than the "te_output")
  • Don't delete destination outbucket prior to copying in smartflow egress (A potentially breaking change)

Checklist

# should be run before merging:
cd watch/
./run_developer_setup.sh  # make sure you're in a virtualenv
python dev/lint.py directory_i_changed/ [--mode=apply to fix]
python run_tests.py
  • New code is covered by tests
  • New code is documented
  • Linting passes
  • Tests pass
  • This branch is rebased on master [maintainer]

Merge request reports