Commit 70e96522 authored May 07, 2026 by Andrey Filippov

Claude: Fix filter5Targets select_new silently forced to false when motion_sequence=null



Line 8734 was: select_new = (motion_sequence != null) ? select_new_in : false
This caused pass 2 (filter5(null, tsm, false, true, ...)) to enter the
!select_new path — finding best existing good tiles instead of untested
non-centered tiles — so nothing was ever selected and the centered-mode
loop was a no-op.

Fix: decouple select_new from motion_sequence null-check; only use_motion
is forced false when motion_sequence is null.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

parent c9316c55

src/main/java/com/elphel/imagej/cuas/CuasMotion.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -8731,7 +8731,7 @@ public class CuasMotion {
		final int [] passes, // debugging - number of passes required
		final int debugLevel){
		final boolean use_motion = (motion_sequence != null) ? use_motion_in : false;
		final boolean select_new = (motion_sequence != null) ? select_new_in : false;
		final boolean select_new = select_new_in; // By Claude on 05/07/2026: decouple from motion_sequence so null works with select_new=true
		boolean debug_now = (debugLevel<5);
		final int num_seq = target_sequence_multi.length;
		final int num_tiles = target_sequence_multi[0].length;

Admin message