Merge pull request #70 from pllim/stop-gap

Raise exception when catalog gap detected

pysynphot/catalog.py

@@ -204,6 +204,11 @@ def _getSpectrum(self, parlist, basename):
                                                   os.path.join(basename,filename))
         sp = spectrum.TabularSourceSpectrum(filename, fluxname=column)
 
+        totflux = sp.integrate()
+        if not N.isfinite(totflux) or totflux <= 0:
+            raise exceptions.ParameterOutOfBounds(
+                "Parameter '{0}' has no valid data.".format(parlist))
+
         result = []
         for member in parlist:
             result.append(member)

pysynphot/test/test_catalog.py

@@ -66,13 +66,13 @@ def test_flux(self):
 
 @use_cdbs
 @pytest.mark.parametrize(
-  ('teff', 'z', 'logg'),
-  [(6440, 0, 10),
-   (6440, 0, -1),
-   (1.e6, 0, 4.3),
-   (100, 0, 4.3),
-   (6440, 2, 4.3),
-   (6440, -6, 4.3)])
+    ('teff', 'z', 'logg'),
+    [(6440, 0, 10),
+     (6440, 0, -1),
+     (1.e6, 0, 4.3),
+     (100, 0, 4.3),
+     (6440, 2, 4.3),
+     (6440, -6, 4.3)])
 def test_Icat_exceptions(teff, z, logg):
     """
     Tests for changes related to Trac ticket #211.
@@ -82,6 +82,18 @@ def test_Icat_exceptions(teff, z, logg):
         Icat('k93models', teff, z, logg)
 
 
+@use_cdbs
+def test_phoenix_gap():
+    """
+    https://github.com/spacetelescope/pysynphot/issues/68
+    """
+    Icat('phoenix', 2700, -1, 5.1)  # OK
+    with pytest.raises(ParameterOutOfBounds):
+        Icat('phoenix', 2700, -0.5, 5.1)
+    with pytest.raises(ParameterOutOfBounds):
+        Icat('phoenix', 2700, -0.501, 5.1)
+
+
 @use_cdbs
 class TestCatalogCache(object):
     """