<div dir="ltr">Hello Hy,<div><br></div><div>Good catch. I'm not sure what's the best way to avoid this.. I mean, the state machine change is correct, but FLIP_PLANNED is called before focus is done and guiding is resumed, correct? </div><div><br></div><div>That variable is set to make sure we are indeed already guiding. What if guiding was never utilized when capture was started? I recall that before we set this immediately after you start a sequence, so we know whether guiding is used or not. But later on, it was moved to check for these conditions later on.</div><div><br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>--</div><div>Best Regards,<br>Jasem Mutlaq<br></div><div><br></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 3, 2020 at 10:06 PM Hy Murveit <<a href="mailto:murveit@gmail.com">murveit@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I just saw a bug where, <div><br></div><div>meridian flip is about to got to FLIP_PLANNED</div><div><br></div><div>capture starts a focus on filter change, so guiding goes from guiding to suspended</div><div><br></div><div>capture gets the FLIP_PLANNED, but because guiding is suspended, it sets </div><div>resumeGuidingAfterFlip to false.</div><div>  capture.cpp line 3636 resumeGuidingAfterFlip = isGuidingActive();<br></div><div><br></div><div>then focus finishes, guiding is resumed, then eventually flip happens, but because the variable is set that way, guiding doesn't get resumed.</div><div><br></div><div>Do you think it's important to not resume guiding on a suspended? I would think, pretty much any time we're in suspended then flip, we'd want to resume guiding after the flip.</div><div><br></div><div>Hy</div><div><br></div></div>
</blockquote></div>