[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kstars-devel
Subject:    [Kstars-devel]
From:       Akarsh Simha <akarshsimha () gmail ! com>
Date:       2008-06-29 20:53:48
Message-ID: 1214772828.755844.13539.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 826104 by asimha:

Avoiding hard-coding the magnitude to which the catalog covers all
stars, by making a guesstimate from the faintest star listed in the
catalog.

CCMAIL: kstars-devel@kde.org



 M  +5 -3      starcomponent.cpp  


--- branches/kstars/summer/kdeedu/kstars/kstars/skycomponents/starcomponent.cpp \
#826103:826104 @@ -259,7 +259,7 @@
     visibleStarCount = 0;
 
     t.start();
-    float fake_maglim = ( ( maglim > 13.0 ) ? 13.0 : maglim );
+    float fake_maglim = ( ( maglim > m_FaintMagnitude * (1 - 1.5/16) ) ? \
m_FaintMagnitude * ( 1 - 1.5/16 ) : maglim );  
     if( veryFrugalMem )
         m_StarBlockFactory.freeAll();
@@ -326,8 +326,10 @@
         }
         t_drawNamed += t.restart();
 
-        // TODO: Hardcoded star catalog faint limit
-        if( !m_starBlockList[ currentRegion ]->fillToMag( maglim ) && maglim <= 12.0 \
) { +        // NOTE: We are guessing that the last 1.5/16 magnitudes in the catalog \
are just additions and the star catalog +        //       is actually supposed to \
reach out continuously enough only to mag m_FaintMagnitude * ( 1 - 1.5/16 ) +        \
// TODO: Is there a better way? We may have to change the magnitude tolerance if the \
catalog changes +        if( !m_starBlockList[ currentRegion ]->fillToMag( maglim ) \
                && maglim <= m_FaintMagnitude * ( 1 - 1.5/16 ) ) {
             kDebug() << "SBL::fillToMag( " << maglim << " ) failed for trixel " 
                      << currentRegion << " !"<< endl;
         }
_______________________________________________
Kstars-devel mailing list
Kstars-devel@kde.org
https://mail.kde.org/mailman/listinfo/kstars-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic