Skip to content

Fix installations for Python 3.12#164

Merged
urinieto merged 4 commits into
urinieto:mainfrom
carlthome:patch-3
Jul 9, 2025
Merged

Fix installations for Python 3.12#164
urinieto merged 4 commits into
urinieto:mainfrom
carlthome:patch-3

Conversation

@carlthome
Copy link
Copy Markdown
Contributor

Need to replace imp with importlib in setup.py for Python 3.12 support (imp has been removed now).

@carlthome
Copy link
Copy Markdown
Contributor Author

carlthome commented Jun 10, 2025

Blocked by marl/jams#225 / marl/jams#224 / marl/jams#223

For Python 3.12 support (imp has been removed)
@urinieto
Copy link
Copy Markdown
Owner

LGTM. Before merging: why are several tests not passing?

@carlthome
Copy link
Copy Markdown
Contributor Author

carlthome commented Jun 29, 2025

There were upstream issues from JAMS with Python 3.12 changes (imp removal was hit there too). Resolved with JAMS 0.3.5 marl/jams#226 so that's great news.

Though there's some additional SciPy errors to fix:

FAILED test_run.py::test_run_algorithms - AttributeError: module 'scipy.signal' has no attribute 'gaussian'

@carlthome
Copy link
Copy Markdown
Contributor Author

Looks good now! Tests passing on Python 3.8 to 3.12. 🎉

@urinieto
Copy link
Copy Markdown
Owner

urinieto commented Jun 30, 2025 via email

@carlthome
Copy link
Copy Markdown
Contributor Author

taking a few days off

good!!

@urinieto urinieto merged commit 577cdba into urinieto:main Jul 9, 2025
6 checks passed
@urinieto
Copy link
Copy Markdown
Owner

urinieto commented Jul 9, 2025

Merged, thanks so much, @carlthome !

@carlthome carlthome deleted the patch-3 branch October 5, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants