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

Python 3.11 #807

Merged
Merged

Conversation

marvinpfoertner
Copy link
Collaborator

@marvinpfoertner marvinpfoertner commented Mar 21, 2023

In a Nutshell

This PR bumps the Python version to 3.11.

Detailed Description

Related Issues

None

@marvinpfoertner marvinpfoertner added testing and CI Unit tests, coverage and continuous integration dependencies Pull requests that update a dependency file labels Mar 21, 2023
@marvinpfoertner marvinpfoertner requested a review from a team as a code owner March 21, 2023 10:36
@marvinpfoertner marvinpfoertner self-assigned this Mar 21, 2023
@marvinpfoertner
Copy link
Collaborator Author

marvinpfoertner commented Mar 21, 2023

@JonathanWenger The tests fail because GPy is not compatible with Python 3.11 (see SheffieldML/GPy#998). How should we address this?

@JonathanWenger
Copy link
Contributor

@JonathanWenger The tests fail because GPy is not compatible with Python 3.11 (see SheffieldML/GPy#998). How should we address this?

As discussed offline, we'll remove the GPy dependency as part of this PR. Since removing it was planned anyway.

@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #807 (e4722eb) into main (2d1f5a0) will decrease coverage by 0.15%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #807      +/-   ##
==========================================
- Coverage   91.29%   91.14%   -0.15%     
==========================================
  Files         218      218              
  Lines        8210     8210              
  Branches     1061     1061              
==========================================
- Hits         7495     7483      -12     
- Misses        487      499      +12     
  Partials      228      228              

see 1 file with indirect coverage changes

@marvinpfoertner marvinpfoertner merged commit 67d7d43 into probabilistic-numerics:main Mar 21, 2023
@marvinpfoertner marvinpfoertner deleted the python-3.11 branch March 21, 2023 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file testing and CI Unit tests, coverage and continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants