LDSO icon indicating copy to clipboard operation
LDSO copied to clipboard
verified publisher icon tum-vision

needNewKFAfter not update

Open match08 opened this issue 5 years ago • 1 comments

needNewKFAfter not update?, if not linearizeOperation must update, dso: if(needKF) needNewKFAfter=fh->shell->trackingRef->id;

match08 avatar Mar 04 '20 00:03 match08

Yes, looks like the flag needNewKFAfter is not really used since it is not updated, but I can't easily tell what are the consequences. Have you analysed in which scenario this causes an issue?

@gaoxiang12 did some refactoring compared to DSO, so the flag might also be obsolete.

NikolausDemmel avatar Mar 23 '20 17:03 NikolausDemmel