Use vmin/vmax offset by eps

matplotlib · tacaswell · Nov 22, 2015 · Nov 20, 2015 · Nov 20, 2015 · Nov 20, 2015
commit 5428cea4c1041b8a9ccad5c1b40186848ad0dedc
diff --git a/lib/matplotlib/contour.py b/lib/matplotlib/contour.py
@@ -1233,9 +1233,9 @@ def _process_levels(self):
        # normed range:
        finfo = np.finfo(float)
        if self.extend in ('both', 'min'):
-            self.layers[0] = finfo.min
+            self.layers[0] = self.vmin - finfo.eps
        if self.extend in ('both', 'max'):
-            self.layers[-1] = finfo.max
+            self.layers[-1] = self.vmax + finfo.eps

    def _process_colors(self):
        """
-Original file line number
+Diff line change
@@ -1233,9 +1233,9 @@ def _process_levels(self):
             # normed range:
             finfo = np.finfo(float)
             Copy link

  
      
    
  

  
      

  
  Member


      

  

  
    
      

      
            tacaswell
  

      

      

      


        Nov 20, 2015


      
    

  


        
      
  
  
  
    

    There was a problem hiding this comment.


  

 
  
    

    Choose a reason for hiding this comment

    
      The reason will be displayed to describe this comment to others. Learn more.
    

    
      
      


  


  
    
      iirc, this can be a major time sink
    
  
  


    

        
      
  
  
    
  
  
  
    
    Sorry, something went wrong.
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.


  
  


          
      
  
    
    
      
        
            
    All reactions
  


          
          
        
      
    

    



    
        
  
    
        
    
  


      
          
  
      
            Copy link

  
      
    
  

  
      

  
  Member


      

  Author


  

  
    
      

      
            mdboom
  

      

      

      


        Nov 20, 2015


      
    

  


        
      
  
  
  
    

    There was a problem hiding this comment.


  

 
  
    

    Choose a reason for hiding this comment

    
      The reason will be displayed to describe this comment to others. Learn more.
    

    
      
      


  


  
    
      Yeah.  In this case, I don't think it's in a hot path, though.
    
  
  


    

        
      
  
  
    
  
  
  
    
    Sorry, something went wrong.
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.


  
  


          
      
  
    
    
      
        
            
    All reactions
             if self.extend in ('both', 'min'):
-                self.layers[0] = finfo.min
+                self.layers[0] = self.vmin - finfo.eps
             if self.extend in ('both', 'max'):
-                self.layers[-1] = finfo.max
+                self.layers[-1] = self.vmax + finfo.eps
         def _process_colors(self):
             """
-          Expand Down