<table><tr><td style="">chrisrowland created this revision.<br />Herald added a project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br />chrisrowland 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/D29549">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Many places in Kstars abd EKOS us precession only to change J2000 catalogue<br />
coordinates to the observed position, omitting nutuation and aberration.<br />
Also the conversion from an observed position to a J2000 catalogue<br />
position was implemented incorrectly, aberration and nutauion were not being removed.</p>

<p>This updates SkyPoint to correct these and adds unit tests to check them.<br />
I also found that the SkyPoint aberration and nutation corrections gave<br />
incorrect results and these have been replaced by the libnova corrections<br />
which testing show are better.  I am using SOFA to generate test positions.</p>

<p>The changes to SkyPoint have been used to implement coordinate conversions<br />
correctly in KStars and EKOS, by replacing precession conversion with<br />
apparentCoord() and CatalogueCoord() as appropriate.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Run the test_skypoint test.  At present two tests fail by a small amount for<br />
declinations close to 90.</p>

<p>Review the changes. I don't think I've made a change which isn't required but<br />
may have missed some.</p>

<p>Build and test. For example the details dialog should now give a J2000<br />
position that matches the J2000 positionin the image header. Make sure that<br />
you have an up to date version if INDI. I had to update that as well.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>SkyPoint_changes (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29549">https://phabricator.kde.org/D29549</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>Tests/skyobjects/test_skypoint.cpp<br />
Tests/skyobjects/test_skypoint.h<br />
kstars/dialogs/focusdialog.cpp<br />
kstars/ekos/mount/mount.cpp<br />
kstars/ekos/scheduler/scheduler.cpp<br />
kstars/hips/hipsrenderer.cpp<br />
kstars/indi/inditelescope.cpp<br />
kstars/kstarsactions.cpp<br />
kstars/skycomponents/flagcomponent.cpp<br />
kstars/skycomponents/skymesh.cpp<br />
kstars/skymap.cpp<br />
kstars/skyobjects/jupitermoons.cpp<br />
kstars/skyobjects/skypoint.cpp<br />
kstars/skyobjects/skypoint.h<br />
kstars/tools/altvstime.cpp<br />
kstars/tools/starhopper.cpp</div></div></div><br /><div><strong>To: </strong>chrisrowland<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>