thin version of is_library, found the guilty one

Signed-off-by: dafnapension <dafnashein@yahoo.com>

Author: dafnapension

src/unitxt/artifact.py

@@ -92,69 +92,69 @@ def is_library_module(module_name):
     ):
         return False
 
-    # if module_name in sys.modules:
-    #     return True
+    if module_name in sys.modules:
+        return True
 
-    # try:
-    #     importlib.import_module(module_name)
-    #     return True
-    # except:
-    #     return False
+    try:
+        importlib.import_module(module_name)
+        return True
+    except:
+        return False
 
-    if module_name not in sys.modules:
-        try:
-            __import__(module_name)
-        except ImportError:
-            return False
+    # if module_name not in sys.modules:
+    #     try:
+    #         __import__(module_name)
+    #     except ImportError:
+    #         return False
 
-    module = sys.modules[module_name]
+    # module = sys.modules[module_name]
 
-    # Built-in modules
-    if not hasattr(module, "__file__") or module.__file__ is None:
-        return True
+    # # Built-in modules
+    # if not hasattr(module, "__file__") or module.__file__ is None:
+    #     return True
 
-    file_path = module.__file__
+    # file_path = module.__file__
 
-    # Check for standard library patterns
-    import sysconfig
+    # # Check for standard library patterns
+    # import sysconfig
 
-    stdlib_path = sysconfig.get_path("stdlib")
+    # stdlib_path = sysconfig.get_path("stdlib")
 
-    # Direct match
-    if file_path.startswith(stdlib_path):
-        return True
+    # # Direct match
+    # if file_path.startswith(stdlib_path):
+    #     return True
 
-    # Handle different Python installations (Homebrew, pyenv, etc.)
-    # Look for common stdlib patterns: .../lib/python3.x/...
-    import re
+    # # Handle different Python installations (Homebrew, pyenv, etc.)
+    # # Look for common stdlib patterns: .../lib/python3.x/...
+    # import re
 
-    if (
-        re.search(r"/lib/python\d+\.\d+/", file_path)
-        and "site-packages" not in file_path
-    ):
-        return True
+    # if (
+    #     re.search(r"/lib/python\d+\.\d+/", file_path)
+    #     and "site-packages" not in file_path
+    # ):
+    #     return True
 
-    # Check if it's an installed package
-    if any(pkg_dir in file_path for pkg_dir in ["site-packages", "dist-packages"]):
-        return True  # Regular installed package
+    # # Check if it's an installed package
+    # if any(pkg_dir in file_path for pkg_dir in ["site-packages", "dist-packages"]):
+    #     return True  # Regular installed package
 
-    # Check if it's an editable install (file outside site-packages but package is installed)
-    site_packages = sysconfig.get_path("purelib")
-    package_name = module_name.split(".")[0]
+    # # Check if it's an editable install (file outside site-packages but package is installed)
+    # site_packages = sysconfig.get_path("purelib")
+    # package_name = module_name.split(".")[0]
 
-    import glob
+    # import glob
 
-    # Check for various editable install patterns
-    egg_links = glob.glob(os.path.join(site_packages, f"{package_name}*.egg-link"))
-    pth_files = glob.glob(os.path.join(site_packages, f"*{package_name}*.pth"))
-    editable_pth = glob.glob(
-        os.path.join(site_packages, f"__editable__.{package_name}*.pth")
-    )
+    # # Check for various editable install patterns
+    # egg_links = glob.glob(os.path.join(site_packages, f"{package_name}*.egg-link"))
+    # pth_files = glob.glob(os.path.join(site_packages, f"*{package_name}*.pth"))
+    # editable_pth = glob.glob(
+    #     os.path.join(site_packages, f"__editable__.{package_name}*.pth")
+    # )
 
-    if egg_links or pth_files or editable_pth:
-        return True  # It's an editable install, still a library
+    # if egg_links or pth_files or editable_pth:
+    #     return True  # It's an editable install, still a library
 
-    return False
+    # return False
 
 
 def import_module_from_file(file_path):

tests/catalog/test_preparation.py

@@ -27,10 +27,6 @@
 # Make sure the order in which the tests are run is deterministic
 # Having a different order for local testing and github testing may cause diffs in results.
 all_preparation_files.sort()
-all_preparation_files = [
-    file for i, file in enumerate(all_preparation_files) if i % 12 == 1
-]
-
 num_par = 1  # num of parallel executions
 logger.critical(
     f"Over all, {len(all_preparation_files)} files will now be tested over {num_par} parallel processes."