[Kstars-devel] branches/kstars/summer/kdeedu/kstars/kstars/skycomponents

[Akarsh Simha]

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 at 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;
         }