Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add python 3.11 support, fix installation and ci #328

Closed

Conversation

jmahlik
Copy link

@jmahlik jmahlik commented Feb 13, 2023

BREAKING CHANGE: drop python 2 support, minimul required python is 3.8+

  • Add official support and testing for python 3.8-3.11.
  • Updated the ci to run on python 3.8-3.11 and removed conda from ci (didn't seem needed).
  • All tests pass locally on python 3.11 without scikit-sparse installed.
  • The ci tests may have a problem on python 3.11 since scikit-sparse doesn't appear to support it. Can be adjusted as needed. Maybe don't install sparse on the python 3.11 tests.
  • Drop support for python 2-3.7. 3.7 is end of life in 4 months.
  • Fix the build system and move to pyproject.toml
  • Restricted numpy version until it is fixed by Compatibility with numpy 1.24 #325.

Closes #319 #312

@dswah would you mind having a look at this PR? pyGAM is having some difficulties with newer versions of python and numpy.

It looks like there hasn't been a release in quite some time. Wondering if you still have interest in maintaining this project? I am willing to help maintain the project if possible. Let me know thoughts :).

BREAKING CHANGE: drop python 2 support, minimul required python is 3.8+
@miguelfmc
Copy link

Are there any plans to accept this PR?

Echoing what @jmahlik said - it would be great to have pyGAM work smoothly with more recent Python versions.

I'm also happy to help with the project although I'm quite the newbie when it comes to open source.

@Zethson
Copy link

Zethson commented Dec 19, 2023

@dswah are you willing to give someone else maintainer rights so that the community can take this over?

@dswah
Copy link
Owner

dswah commented Feb 8, 2024

Yes i am happy to give the community maintainer rights!
@Zethson

@Zethson
Copy link

Zethson commented Feb 11, 2024

Maintainer question aside, I don't get why you closed instead of merging this PR

@dswah
Copy link
Owner

dswah commented Feb 11, 2024

I didnt merge because the main functionality from this PR has been in production since March 2023 after the merge of PR #332

Admittedly, I should have had a conversation with the original contributor a month before that, because there were a few issues with this PR (that were addressed in #332). Specifically:

  • i had been wanting to migrate to poetry for a while because the improved dependency solving removes a big headache
  • this PR depends on Travis CI, which is no longer free

I'm sorry @jmahlik because I never commented on your PR when the time was appropriate, I didn't thank you for your contribution, and I closed this PR a year later with no comment to you personally. Your initiative was 100% correct. I acted wrongly and am sorry.

@jmahlik
Copy link
Author

jmahlik commented Feb 15, 2024

No worries @dswah. I saw the other PR so should have closed this one, just didn't get around to clicking the button in time.

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.

Can't use flit to install pygam - pyproject.toml does not exist
4 participants