Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

Incompatible with Python 3.12 #52

Open
pseudo-rnd-thoughts opened this issue May 21, 2024 · 2 comments
Open

Incompatible with Python 3.12 #52

pseudo-rnd-thoughts opened this issue May 21, 2024 · 2 comments

Comments

@pseudo-rnd-thoughts
Copy link

BSuite uses imp which is deprecated in favour of importlib and removed in 3.12
https://docs.python.org/3.11/library/imp.html

@bionicles
Copy link

same issue today

@bionicles
Copy link

> rg 'imp\.'
setup.py
70:    version=imp.load_source('_metadata', 'bsuite/_metadata.py').__version__,

might be easy to fix, i only see one spot

workaround, you can clone the repo and edit setup.py

# import imp

from typing import Optional, Tuple
import os
import re


def extract(path: Optional[str] = None) -> str:
    assert path is not None
    assert os.path.exists(path)
    try:
        with open(path, "r") as f:
            content = f.read()
        return content
    except Exception as e:
        print("extract Exception", e)
        raise e


# imp.load_source('_metadata', 'bsuite/_metadata.py').__version__
def version_from_path(source_path: Optional[str] = None) -> Tuple[int, int, int]:
    assert source_path is not None
    assert source_path.endswith(".py")
    assert os.path.exists(source_path)
    content = extract(source_path)
    if match := re.search('\n__version__ = "(.*)"', content):
        version = match.group(1).lstrip("\n")
        major, minor, patch = map(int, version.split("."))
        return major, minor, patch
    raise ValueError("Failed to match a version")


METADATA_PATH = os.path.join("bsuite", "_metadata.py")
major, minor, patch = version_from_path(METADATA_PATH)
version = f"{major}.{minor}.{patch}"

this could be done a billion different ways but that's how I extract versions for "tree_plus"

later, in the call to setuptools.setup:

-     version=imp.load_source('_metadata', 'bsuite/_metadata.py').__version__,
+    version=version,

works on my machine :o

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants