Skip to content

Conversation

@bjoaofelipe
Copy link
Contributor

No description provided.

@bjoaofelipe bjoaofelipe linked an issue Sep 22, 2025 that may be closed by this pull request
Copy link
Contributor

@MattScicluna MattScicluna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you can merge as-is, but left some comments for your consideration.

]
license = {text = "GNU General Public License Version 2"}
readme = "README.md"
requires-python = ">=3.9"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought this works for python >=3.8 ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should make sure this doesnt break the phate tutorial notebook

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we not need to test in python 3.11+ ?

@bjoaofelipe
Copy link
Contributor Author

I have removed 3.8 since it is already in end-of-life. Added tests for the other two's.
We need to update scprep in order to get phate tutorial to perform as expected.

@bjoaofelipe bjoaofelipe merged commit 3b68c3c into main Sep 22, 2025
12 checks passed
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.

Update requirements and installation procedure.

3 participants