<table><tr><td style="">lancaster added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24257">View Revision</a></tr></table><br /><div><div><p>Adding comments</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D24257#inline-137320">View Inline</a><span style="color: #4b4d51; font-weight: bold;">TallFurryMan</span> wrote in <span style="color: #4b4d51; font-weight: bold;">guide.cpp:548</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Switch case might have been appropriate now that all cases are distinct.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Well my hope is to get rid of the if statement that now only happens for Linguider entirely.  But I don't have Linguider to try to make the same changes that I did for PHD2.  Nor do I have any experience with it.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D24257#inline-137321">View Inline</a><span style="color: #4b4d51; font-weight: bold;">TallFurryMan</span> wrote in <span style="color: #4b4d51; font-weight: bold;">guide.cpp:595</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">As a side note, this will be the case in my obs, so that branch will tested in my environment. I think the option "receive external guide frames" is confusing and should be renamed (no action from your part, warning is OK).</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">So I partially agree.  If you have seen my comments before, I have definitely advocated using PHD2-only connected guiders rather than using INDI connected guiders that are also connected to Ekos.  In fact this is what I have been doing on my system the last couple of years.</p>

<p style="padding: 0; margin: 8px;">I would like to see this option of switching back and forth between receiving the full guide frames and the profile image become an option similar to and as easy to use as the "use subframe" checkboxes in Ekos.  This is like my first step on that road.  It still requires reconnecting or restarting.  I haven't yet figured out how to make it switch perfectly.  The issue is that the guide camera getting blobs enabled and disabled doesn't always seem to work except for the first time I call it.  So a reconnection or restart is needed?  it needs further analysis.  But for now, the message just says restart Ekos and that works.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24257">https://phabricator.kde.org/D24257</a></div></div><br /><div><strong>To: </strong>lancaster, mutlaqja, alexcherney, TallFurryMan<br /><strong>Cc: </strong>hansl, wreissenberger, kde-edu, narvaez, apol<br /></div>