[Kstars-devel] Re: Review Request: This patch fixes the magnitude parameter of observable comets by using a different formula altogether.

[Samikshan Bairagya]

> On June 17, 2011, 5:53 a.m., Akarsh Simha wrote:
> > kstars/skyobjects/kscomet.h, line 111
> > <http://git.reviewboard.kde.org/r/101650/diff/1/?file=24596#file24596line111>
> >
> >     Don't you have methods to return M1 and M2? Are those not required?

Okay I'll add those. 


- Samikshan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101650/#review3946
-----------------------------------------------------------


On June 17, 2011, 5:15 a.m., Samikshan Bairagya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101650/
> -----------------------------------------------------------
> 
> (Updated June 17, 2011, 5:15 a.m.)
> 
> 
> Review request for KStars.
> 
> 
> Summary
> -------
> 
> The patch is dirty now. Will look into it. This patch fixes the magnitude parameter of observable comets by using a different formula altogether.
> T-Mag = M1 + 5*log10(rearth) + k1*log10(rsun). Comets for which data are not available will have abnormal magnitudes of 99.
> For formula refer: http://ssd.jpl.nasa.gov/horizons.cgi
> 
> Changed CometsComponent::updateDataFile() to store the k1,k2 parameters as well in the comets.dat file
> Changed CometsComponent::loadData() to load M1, M2, K1, K2 parameters as well. Also if M1, M2, K1, K2 data are not available set abnormal values as default.
> Changed KSComet.h and KSComet.cpp to accept M1, M2, K1, K2 parameters as well. Also modified KSComet::findMagnitude() to use the above formula.
> Changed setMag() function in SkyObject.h so that for any magnitudes >=30 abnormally high value(99) is set. 
> 
> 
> Diffs
> -----
> 
>   kstars/dialogs/detaildialog.cpp 3d669ef 
>   kstars/skycomponents/cometscomponent.cpp 4a2851b 
>   kstars/skyobjects/kscomet.h 5a858ee 
>   kstars/skyobjects/kscomet.cpp e665c70 
>   kstars/skyobjects/skyobject.h 6f6a061 
> 
> Diff: http://git.reviewboard.kde.org/r/101650/diff
> 
> 
> Testing
> -------
> 
> Tested for various comets.
> 
> 
> Thanks,
> 
> Samikshan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kstars-devel/attachments/20110618/9252a2e4/attachment.htm