Skip to content

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

Open
hatzel wants to merge 1 commit intoforTEXT:mainfrom
hatzel:fix/setup_py
Open

Ensure setup.py compatiblity with recent Python versions.#11
hatzel wants to merge 1 commit intoforTEXT:mainfrom
hatzel:fix/setup_py

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