[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-edu-devel
Subject:    D18510: Ring-field focusing feature.
From:       Eric Dejouhanet <noreply () phabricator ! kde ! org>
Date:       2019-01-25 8:46:27
Message-ID: 9a29bb30fd863934858faa08ab46b101 () localhost ! localdomain
[Download RAW message or body]

[Attachment #2 (text/plain)]

TallFurryMan created this revision.
TallFurryMan added reviewers: mutlaqja, wreissenberger.
Herald added a project: KDE Edu.
Herald added a subscriber: kde-edu.
TallFurryMan requested review of this revision.

REVISION SUMMARY
  Adds inner and outer limits to filter stars when running a full-frame \
focus procedure.  When option "Full Field" is enabled in the Focus tab, the \
procedure will first detect  stars that are visible on the frame, using \
either the SEP method (if selected) or the  Centroid method, and remove \
stars not in the ring surface bounded by the inner and  outer radius values \
before calculating the mean HFR.

TEST PLAN
  Basic items:
  
  - Default options are "Sub Frame", and inner/outer radius boxes disabled.
  - Selecting "Full Frame" enables the radius boxes, default values are 0% \
                and 100%, respectively.
  - Tooltips are coherent, and match their gadgets.
  - Enabling "Sub Frame" disables "Full Frame", and enabling "Full Frame" \
                disables "Sub Frame".
  - Procedure values and selection are remembered from one execution of \
KStars to the other.  
  Procedure:
  
  - Filter use is enabled with option "Full Frame".
  - Filter is inoperant if inner/outer radius value are 0% and 100%, \
                respectively.
  - If one of the radius values is not the default, the filter is used.
  - When the filter is used, two blue dashed circles are displayed on the \
                focus frame (values 0% and 100% are not visible).
  - Stars inside the inner circle are disregarded for the focus procedure, \
                and are not marked by the star overlay.
  - Stars outside the outer circle are disregarded for the focus procedure, \
and are not marked by the star overlay.

REPOSITORY
  R321 KStars

BRANCH
  feature__ring_frame_focusing (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18510

AFFECTED FILES
  kstars/ekos/focus/focus.cpp
  kstars/ekos/focus/focus.ui
  kstars/fitsviewer/fitsdata.cpp
  kstars/fitsviewer/fitsdata.h
  kstars/fitsviewer/fitsview.cpp
  kstars/fitsviewer/fitsview.h
  kstars/kstars.kcfg

To: TallFurryMan, mutlaqja, wreissenberger
Cc: kde-edu, narvaez, apol


[Attachment #3 (text/html)]

<table><tr><td style="">TallFurryMan created this revision.<br \
/>TallFurryMan added reviewers: mutlaqja, wreissenberger.<br />Herald added \
a project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br \
/>TallFurryMan requested review of this revision. </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/D18510">View \
Revision</a></tr></table><br /><div><strong>REVISION \
SUMMARY</strong><div><p>Adds inner and outer limits to filter stars when \
running a full-frame focus procedure.<br /> When option &quot;Full \
Field&quot; is enabled in the Focus tab, the procedure will first detect<br \
/> stars that are visible on the frame, using either the SEP method (if \
selected) or the<br /> Centroid method, and remove stars not in the ring \
surface bounded by the inner and<br /> outer radius values before \
calculating the mean HFR.</p></div></div><br /><div><strong>TEST \
PLAN</strong><div><p>Basic items:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Default options are &quot;Sub Frame&quot;, \
and inner/outer radius boxes disabled.</li> <li \
class="remarkup-list-item">Selecting &quot;Full Frame&quot; enables the \
radius boxes, default values are 0% and 100%, respectively.</li> <li \
class="remarkup-list-item">Tooltips are coherent, and match their \
gadgets.</li> <li class="remarkup-list-item">Enabling &quot;Sub Frame&quot; \
disables &quot;Full Frame&quot;, and enabling &quot;Full Frame&quot; \
disables &quot;Sub Frame&quot;.</li> <li \
class="remarkup-list-item">Procedure values and selection are remembered \
from one execution of KStars to the other.</li> </ul>

<p>Procedure:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Filter use is enabled with option &quot;Full \
Frame&quot;.</li> <li class="remarkup-list-item">Filter is inoperant if \
inner/outer radius value are 0% and 100%, respectively.</li> <li \
class="remarkup-list-item">If one of the radius values is not the default, \
the filter is used.</li> <li class="remarkup-list-item">When the filter is \
used, two blue dashed circles are displayed on the focus frame (values 0% \
and 100% are not visible).</li> <li class="remarkup-list-item">Stars inside \
the inner circle are disregarded for the focus procedure, and are not \
marked by the star overlay.</li> <li class="remarkup-list-item">Stars \
outside the outer circle are disregarded for the focus procedure, and are \
not marked by the star overlay.</li> </ul></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R321 \
KStars</div></div></div><br \
/><div><strong>BRANCH</strong><div><div>feature__ring_frame_focusing \
(branched from master)</div></div></div><br /><div><strong>REVISION \
DETAIL</strong><div><a \
href="https://phabricator.kde.org/D18510">https://phabricator.kde.org/D18510</a></div></div><br \
/><div><strong>AFFECTED \
FILES</strong><div><div>kstars/ekos/focus/focus.cpp<br /> \
kstars/ekos/focus/focus.ui<br /> kstars/fitsviewer/fitsdata.cpp<br />
kstars/fitsviewer/fitsdata.h<br />
kstars/fitsviewer/fitsview.cpp<br />
kstars/fitsviewer/fitsview.h<br />
kstars/kstars.kcfg</div></div></div><br /><div><strong>To: \
</strong>TallFurryMan, mutlaqja, wreissenberger<br /><strong>Cc: \
</strong>kde-edu, narvaez, apol<br /></div>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic