Add/subtract one sidereal year jumps wrong amount

Bug #1006208 reported by norbinz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Stellarium
Fix Released
Low
Alexander Wolf

Bug Description

I'm running Version 0.11.3 for Windows.

Pressing Ctrl+Alt+Shift+] should advance time by one sidereal year, but it's off by about a day.

In StelCore.cpp I see this suspicious code:

void StelCore::addSiderealYear()
{
    addSiderealDays(365.256363004);
}

I think that should be addSolarDays, not addSiderealDays. (Or change the value accordingly.)

Similar problem in StelCore::subtractSiderealYear()

Related branches

Revision history for this message
norbinz (norbinz) wrote :

Change last line to:

Similar problem in StelCore::subtractSiderealYear()

norbinz (norbinz)
description: updated
Revision history for this message
Alexander Wolf (alexwolf) wrote :

Why do you think that the length of the sidereal year the invalid?

Revision history for this message
Alexander Wolf (alexwolf) wrote :

The question is removed, the issue is correct.

Changed in stellarium:
status: New → Confirmed
assignee: nobody → Alexander Wolf (alexwolf)
importance: Undecided → Low
milestone: none → 0.11.4
status: Confirmed → In Progress
Revision history for this message
Alexander Wolf (alexwolf) wrote :
Changed in stellarium:
status: In Progress → Fix Committed
Changed in stellarium:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.