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

Ensure setup.py compatiblity with recent Python versions. #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hatzel
Copy link
Contributor

@hatzel hatzel commented Sep 13, 2024

I do not know why the cvxopt version was pinned, but due to changes in configparser (a standard library module), the required version can no longer be installed with recent Python versions.

In my limited testing, I did not encounter any issues with this patch.

@maltem-za
Copy link
Member

Thanks - cvxopt is a dependency of pygamma-agreement (not installed by default except for in the Docker image) and was pinned because we ran into some kind of compatibility issue. The Python version was pinned for a similar reason, although I can't recall now if it was also related to pygamma-agreement. We need to test everything with the latest versions and see if we can remove these restrictions (and possibly also remove cvxopt from setup.py). Will leave this open until we get a chance to do that.

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