Merge pull request #60 from rapidsai/wence/fix/index-again

Fix index `__new__` again

python/cudf/cudf/pandas/_wrappers/pandas.py

@@ -10,7 +10,6 @@
 # its affiliates is strictly prohibited.
 
 import sys
-from typing import Iterator
 
 import pandas as pd
 
@@ -150,28 +149,16 @@ def _DataFrame__dir__(self):
 
 
 def Index__new__(cls, *args, **kwargs):
-    # Avoid consuming generators twice
-    # This is a "good enough" solution for this specific use case.
-    args = tuple(
-        list(arg) if isinstance(arg, Iterator) else arg for arg in args
-    )
-    kwargs = {
-        k: list(v) if isinstance(v, Iterator) else v for k, v in kwargs.items()
-    }
-    # Make the object
+    # Call fast/slow constructor
+    # This takes care of running __init__ as well, but must be paired
+    # with a removal of the defaulted __init__ that
+    # make_final_proxy_type provides.
     self, _ = _fast_slow_function_call(
-        lambda cls, *args, **kwargs: cls.__new__(cls, *args, **kwargs),
+        lambda cls, *args, **kwargs: cls(*args, **kwargs),
         cls,
         *args,
         **kwargs,
     )
-    # Call init
-    _fast_slow_function_call(
-        lambda self, *args, **kwargs: self.__init__(*args, **kwargs),
-        self,
-        *args,
-        **kwargs,
-    )
     return self
 
 

python/cudf/cudf/pandas/scripts/run-pandas-tests.sh

@@ -92,7 +92,7 @@ EOF
     # Substitute `pandas.tests` with a relative import.
     # This will depend on the location of the test module relative to
     # the pandas-tests directory.
-    for hit in $(find . -iname *.py | xargs grep "pandas.tests" | cut -d ":" -f 1 | sort | uniq); do
+    for hit in $(find . -iname '*.py' | xargs grep "pandas.tests" | cut -d ":" -f 1 | sort | uniq); do
         # Get the relative path to the test module
         test_module=$(echo $hit | cut -d "/" -f 2-)
         # Get the number of directories to go up