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

Support python 3.11 #190

Merged
merged 6 commits into from
Jan 19, 2023
Merged

Support python 3.11 #190

merged 6 commits into from
Jan 19, 2023

Conversation

zhx828
Copy link
Member

@zhx828 zhx828 commented Jan 18, 2023

This fixes #188

@zhx828 zhx828 changed the title Skip token validation Add additional python versions for testing Jan 18, 2023
The yaml considers 3.10 as number and will convert it to 3.1
Few rationals behind removing matplotlib from dependency
- we want to support the latest python version as soon as possible, but matplotlib usually has delay of being compatible
- OncoKBPlots is not actively used internally, therefore we have no checks on whether it's working
- OncoKBPlots actually produces low quality images which team does not like
- we are not going to remove OncoKBPlots.py from the repo but just remove the matplotlib dependency. Anyone wants to use it can still install matplotlib at their well
@zhx828 zhx828 changed the title Add additional python versions for testing Support python 3.11 Jan 18, 2023
@zhx828 zhx828 added the feature Feature tag for release label Jan 18, 2023
The mode 'U' is no longer has any effect in python 3 and 3.11 completely removed the option.
@zhx828 zhx828 merged commit f322ccc into oncokb:master Jan 19, 2023
@zhx828 zhx828 deleted the skip-token-validation branch January 19, 2023 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature tag for release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

open() mode 'rU' invalid mode
2 participants