[Kstars-devel] Asteroid magnitudes

[Jason Harris]

Hi Akarsh,

This patch looks good.  One comment:

+    if ( isMajorPlanet() || type() == 10 )

+    if( type() == 10 ) {

please change "10" in these lines to "SkyObject::ASTEROID"

regards,
Jason

On Tuesday 18 December 2007 08:20:07 am Akarsh Simha wrote:
> Hi all
>
> Please find a patch attached that fixes asteroid magnitudes.
>
> The magnitude set to the absolute magnitude for asteroids directly.
>
> The modified version uses a formula for comet nuclei described here
> on Seichii Yoshida's site:
> http://www.aerith.net/astro/Encke-nucleus.html
> which relates the magnitude to phase, and "orbital" elements
> H (Absolute Magnitude) and G (Slope Parameter).
>
> The patch retrieves 'G' from the data files (which was not used earlier)
> and also defines a modified constructor in KSAsteroid which accepts
> the slope parameter in addition. It also adds a bit to ksplanetbase.cpp
> which implements the magnitude calculations for asteroids.
>
> The estimates are not very good, but they are reasonable. While the
> SB What's Observable site (http://ssd.jpl.nasa.gov/sbwobs.cgi) puts Ceres
> and Vesta at 7.93 and 7.97 respectively, and KStars with the patch puts it
> at 8.3 and 7.9 respectively.
>
> Regards
> Akarsh.



-- 
KStars: KDE Desktop Planetarium
kstars at 30doradus.org