ENH: remove unreachable constructors wrappers from xorbits.pandas & remove lhoestq/test huggingface repo in datasets test (xorbitsai#848)

Author: Dawnfz-Lenfeng

python/xorbits/datasets/backends/huggingface/tests/test_huggingface_dataset.py

@@ -26,14 +26,17 @@
 from ....._mars.tests.core import mock
 from ..from_huggingface import from_huggingface
 
-SAMPLE_DATASET_IDENTIFIER = "lhoestq/test"  # has dataset script
+SAMPLE_DATASET_IDENTIFIER = (
+    "lhoestq/test"  # has dataset script. WARNING: now has been deleted
+)
 SAMPLE_DATASET_IDENTIFIER2 = "lhoestq/test2"  # only has data files
 SAMPLE_DATASET_IDENTIFIER3 = (
     "hf-internal-testing/multi_dir_dataset"  # has multiple data directories
 )
 SAMPLE_DATASET_IDENTIFIER4 = "hf-internal-testing/imagefolder_with_metadata"  # imagefolder with a metadata file outside of the train/test directories
 
 
+@pytest.mark.skip(reason="lhoestq/test repository has been deleted")
 def test_split_arg_required():
     with pytest.raises(Exception, match="split"):
         from_huggingface(SAMPLE_DATASET_IDENTIFIER)
@@ -42,7 +45,7 @@ def test_split_arg_required():
 @pytest.mark.parametrize(
     "path",
     [
-        SAMPLE_DATASET_IDENTIFIER,
+        # SAMPLE_DATASET_IDENTIFIER,
         SAMPLE_DATASET_IDENTIFIER2,
         SAMPLE_DATASET_IDENTIFIER3,
         SAMPLE_DATASET_IDENTIFIER4,
@@ -78,7 +81,7 @@ def test_from_huggingface_file_lock(setup):
 @pytest.mark.parametrize(
     "path",
     [
-        SAMPLE_DATASET_IDENTIFIER,
+        # SAMPLE_DATASET_IDENTIFIER,
         SAMPLE_DATASET_IDENTIFIER2,
     ],
 )
@@ -94,7 +97,7 @@ def test_to_dataframe_execute(setup, path):
 @pytest.mark.parametrize(
     "path",
     [
-        SAMPLE_DATASET_IDENTIFIER,
+        # SAMPLE_DATASET_IDENTIFIER,
         SAMPLE_DATASET_IDENTIFIER2,
     ],
 )
@@ -114,7 +117,7 @@ def add_prefix(x):
 @pytest.mark.parametrize(
     "path",
     [
-        SAMPLE_DATASET_IDENTIFIER,
+        # SAMPLE_DATASET_IDENTIFIER,
         SAMPLE_DATASET_IDENTIFIER2,
     ],
 )

python/xorbits/pandas/mars_adapters/core.py

@@ -47,28 +47,6 @@
 def _collect_module_callables() -> Dict[str, Callable]:
     mars_dataframe_callables = dict()
 
-    # install class constructors.
-    mars_dataframe_callables[mars_dataframe.DataFrame.__name__] = wrap_mars_callable(
-        mars_dataframe.DataFrame,
-        attach_docstring=True,
-        is_cls_member=False,
-        docstring_src_module=pandas,
-        docstring_src=pandas.DataFrame,
-    )
-    mars_dataframe_callables[mars_dataframe.Series.__name__] = wrap_mars_callable(
-        mars_dataframe.Series,
-        attach_docstring=True,
-        is_cls_member=False,
-        docstring_src_module=pandas,
-        docstring_src=pandas.Series,
-    )
-    mars_dataframe_callables[mars_dataframe.Index.__name__] = wrap_mars_callable(
-        mars_dataframe.Index,
-        attach_docstring=True,
-        is_cls_member=False,
-        docstring_src_module=pandas,
-        docstring_src=pandas.Index,
-    )
     # install module functions
     for name, func in inspect.getmembers(mars_dataframe, inspect.isfunction):
         mars_dataframe_callables[name] = wrap_mars_callable(