Bug 171990

Summary:	Allow sidereal day time step unit
Product:	[Applications] kstars	Reporter:	Nicolas Dietrich <nidi>
Component:	general	Assignee:	kstars
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	akarsh.simha, mboquien
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Debian testing
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:
Attachments:	Simple patch replacing 12 hours time step by the siderial day.

Description Nicolas Dietrich 2008-10-01 22:06:49 UTC

Version:           1.4.1 (using KDE 4.1.1)
OS:                Linux
Installed from:    Debian testing/unstable Packages

It'd be a useful addition to allow the animation time step to be set to a sidereal day (23h, 56min, ...). 

Doing that one can observe the standstill of the stars while the solar system objects move.

This could be achieved by adding a separate unit 'sidereal day' between hour and day in the time step control.

Comment 1 Nicolas Dietrich 2008-10-01 23:42:37 UTC

Created attachment 27648 [details]
Simple patch replacing 12 hours time step by the siderial day.

A trivial fix is replacing another time step (e.g. 12 hours, which is not very interesting) by the siderial day.

This can be done by applying the attached patch to widgets/timespinbox.cpp of the kdeedu 4.1.1 source (haven't checked trunk). 

This isn't particularly pretty but does it for me.

Comment 2 Médéric Boquien 2009-01-11 00:33:37 UTC

SVN commit 909033 by asimha:

Applying Nico Dietrich's patch (modified) to add a Sidereal Day
time-step to the Time Spin Box.

Nico's patch would replace the 12 hour timestep with a sidereal day
timestep. Instead of doing that, I modified the patch to retain the 12
hour timestep and add a sidereal day timestep as well.

Thanks Nico! This closes wishlist bug #170583.
(Thanks Médéric, for pointing this out)

BUG: 170583
CCMAIL: kstars-devel@kde.org



 M  +23 -21    timespinbox.cpp  
 M  +1 -1      timespinbox.h

Comment 3 Nicolas Dietrich 2009-01-11 00:43:25 UTC

Thanks!