kwiver icon indicating copy to clipboard operation
kwiver copied to clipboard

Porting tests from initialize_cameras_landmarks_basic to test initialize_cameras_landmarks

Open OwenMcGee opened this issue 3 years ago • 1 comments

Currently doesn't compile due to issues in the evaluate_initializer() helper function. With the lines calling that function commented out, all tests currently run and succeed aside from subset_init, which I haven't yet altered.

The helper function helper() (to be renamed) condenses repeat code from the original initialize_cameras_landmarks_basic tests. I also adjusted the configure_algo() helper function to set additional config values that weren't needed in the original basic tests.

OwenMcGee avatar Jun 03 '22 02:06 OwenMcGee

Tests currently fail in the evaluate_initialization helper function, as the est_cams variable is a nullptr. The variable should be set in the initialize function in initialize_cameras_landmarks.cxx, but isn't due to functions returning too early (these checks should probably throw exceptions, rather than just returning early)

OwenMcGee avatar Jun 10 '22 17:06 OwenMcGee