[Kstars-devel] KDE/kdeedu/kstars/kstars/skycomponents

[Jason Harris]

Hi Akarsh,

On Sun, Jul 20, 2008 at 10:50 AM, Akarsh Simha <akarshsimha at gmail.com> wrote:
> -    float maglim = 4.444 * ( lgz - lgmin ) + 2.222 * log10( Options::starDensity() ) + 3.5;
> +    //    float maglim = 4.444 * ( lgz - lgmin ) + 2.222 * log10( Options::starDensity() ) + 3.5;
>
> +    // Reducing the slope w.r.t zoom factor to avoid the extremely fast increase in star density with zoom
> +    // that 4.444 gives us (although that is what the derivation gives us)
> +
> +    float maglim = 3.7 * ( lgz - lgmin ) + 2.222 * log10( Options::starDensity() ) + 3.5;
> +
>     m_zoomMagLimit = maglim;
>
>     double maxSize = 10.0;

There is code in skymapevents.cpp that I suspect needs to be updated
to match these changes.  The functions I am thinking of are
incMagLimit() and decMagLimit(), which are tied to Alt+Mousewheel
events.  Actually, I don't see any dependence on zoom level in those
functions, so I may be wrong that these changes should manifest there
as well.

Anyway, it seems a bit fragile to have similar code in different
places.  Maybe we need SDtoMagLim() and magLimtoSD() functions...