Remove bundled pvextractor and updated extraction tool to work properly with changes to .coords

CHANGES.md

@@ -1,6 +1,12 @@
 Full changelog
 ==============
 
+v1.3.0 (unreleased)
+-------------------
+
+* Remove bundled version of pvextractor and include it as a proper
+  dependency. [#2252]
+
 v1.2.4 (2022-01-27)
 -------------------
 

glue/core/tests/test_state_objects.py

@@ -4,7 +4,6 @@
 
 from echo import CallbackProperty, ListCallbackProperty
 from glue.core import Data, DataCollection
-from glue.utils import nanmedian
 
 from .test_state import clone
 from ..state_objects import (State, StateAttributeLimitsHelper,
@@ -202,7 +201,7 @@ class SimpleState(State):
         self.state = SimpleState()
 
         self.helper = StateAttributeSingleValueHelper(self.state, attribute='comp',
-                                                      function=nanmedian, value='val')
+                                                      function=np.nanmedian, value='val')
 
         self.state.data = self.data
 

glue/core/util.py

@@ -11,8 +11,6 @@
 from matplotlib.projections.polar import ThetaFormatter, ThetaLocator
 import matplotlib.ticker as mticker
 
-from glue.utils import nanmin, nanmax
-
 __all__ = ["ThetaRadianFormatter", "relim", "split_component_view", "join_component_view",
            "facet_subsets", "colorize_subsets", "disambiguate",
            'small_view', 'small_view_array', 'visible_limits',
@@ -205,9 +203,9 @@ def facet_subsets(data_collection, cid, lo=None, hi=None, steps=5,
             raise ValueError("Cannot infer data limits for ComponentID %s"
                              % cid)
         if lo is None:
-            lo = nanmin(vals)
+            lo = np.nanmin(vals)
         if hi is None:
-            hi = nanmax(vals)
+            hi = np.nanmax(vals)
 
     reverse = lo > hi
     if log:
@@ -362,7 +360,7 @@ def visible_limits(artists, axis):
     if data.size == 0:
         return
 
-    lo, hi = nanmin(data), nanmax(data)
+    lo, hi = np.nanmin(data), np.nanmax(data)
     if not np.isfinite(lo):
         return