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
Support Python 3.11 #5994
Support Python 3.11 #5994
Conversation
@bp-kelley : this is an alternate approach to handle the deprecation of the extra argument to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I’ll be able to take a look this weekend. If it’s better Han the previous random fix, I’m all for it. |
@@ -18,3 +17,30 @@ | |||
random = _random.random | |||
randrange = _random.randrange | |||
seed = _random.seed | |||
|
|||
if sys.hexversion >= 0x30b0000: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice fix.
LGTM |
* remove some more deprecated numpy stuff * workaround for changes to random.shuffle in python 3.11 * fix pickling of rdkit mols in python 3.11 * add py311 build to CI * update py311 CI * remove qt for py311 for the moment * only use the new code with pyversion >=3.11 * use the new logic for all pickle_suites * need to work with older py too
This required minor changes to the way pickling is handled and a switch away from using
random.shuffle
with therandom
keyword.Also adds a CI build for py 3.11
Fixes #5911