Bug 400580 - Detail window has incorrect transit time
Summary: Detail window has incorrect transit time
Status: RESOLVED FIXED
Alias: None
Product: kstars
Classification: Applications
Component: general (show other bugs)
Version: git
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Jasem Mutlaq
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-02 09:20 UTC by TallFurryMan
Modified: 2021-09-14 10:09 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 3.5.5
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description TallFurryMan 2018-11-02 09:20:35 UTC
Detail window has incorrect transit time


STEPS TO REPRODUCE
1. Stop simulation, set geographic location to 48°13/-1°42, time and date to 2018/11/02 02:16am
2. Check south, locate Zaurak in Eridanus, open details for this object
3. Open position tab in the detail pop-up

OBSERVED RESULT
Transit time is indicated at 02:16am, but Zaurak Hour Angle is 4'52 minutes east of local meridian at that moment.

EXPECTED RESULT
Transit time indicated at 02:20am.

SOFTWARE VERSIONS
(available in About System)
KDE Plasma Version: -
KDE Frameworks Version: 5.38.0
Qt Version: 5.9.1

ADDITIONAL INFORMATION
Comment 1 TallFurryMan 2018-11-02 09:25:35 UTC
I found this while rewriting the planning algorithm of the Scheduler.

#267265 is similar, but fixes an issue deeper in the calculations.
Calculating transit time with KNumbers gives proper results.
The issue seems to be in the way the details pop-up consolidates the target coordinates. The call to updateCoords is dubious.

Tested with Stellarium supports the issue claim, as in the same geographic and time conditions transit time is 02:20 there too.
Comment 2 Jasem Mutlaq 2018-11-02 14:29:48 UTC
Okay, any idea on what is causing this?
Comment 3 TallFurryMan 2018-11-02 21:22:20 UTC
No yet. Preliminary guess is that while calculating, the algorithm incorrectly moves to a different day, which offsets the transit time by 4 minutes.
Comment 4 Jasem Mutlaq 2018-11-03 13:08:14 UTC
So I just checked with SkyCharts & Stellarium. There is no consistent results. Time varies anytime between 1-5 minutes. Sometimes, KStars & SkyChart agree but differently from Stellarium, and sometimes it's the opposite, and sometimes SkyCharts & Stellarium agree but different from KStars.

There is no consistency at all for more than 10 different objects tested. It's not simply a matter of 1-day off offset.
Comment 5 TallFurryMan 2018-11-03 13:42:17 UTC
Sure. I was only referring to the 4 minute offset. Actually I do not compare softwares to prove which is right and which is wrong: this is bug is only about the fact that the transit time presented in the dialog doesn't result in the object being at the meridian at that moment. D16429 has an algorithm for culmination that is compatible with what is drawn by KStars, so we could use that to debug the dialog code and see at which point the execution diverges. Now, which computation is the correct one is another story.
Comment 6 TallFurryMan 2018-11-03 13:43:33 UTC
*being at the meridian at that moment on the chart that is drawn on the screen.
Comment 7 Jasem Mutlaq 2021-09-14 07:53:15 UTC
This is quite similar to the moon transit bug, I think the algorithm is faulty.
Comment 8 Jasem Mutlaq 2021-09-14 10:09:02 UTC
Git commit a9b1d5a220a5b038a41f157d5db0be708b68bb23 by Jasem Mutlaq.
Committed on 14/09/2021 at 10:06.
Pushed by mutlaqja into branch 'master'.

Fix transit and set times for several types of objects. Only remaining problematic object is comets. Lunar, Solar, Planetary, and Stellar objects rise, transit, and set times confirmed from multiple sources.
Related: bug 438591
FIXED-IN:3.5.5

M  +46   -45   kstars/dialogs/detaildialog.cpp
M  +7    -10   kstars/skyobjects/skyobject.cpp

https://invent.kde.org/education/kstars/commit/a9b1d5a220a5b038a41f157d5db0be708b68bb23