Skip to content

Conversation

@gregunz
Copy link

@gregunz gregunz commented Sep 26, 2025

Rationale for this change

Updated Ray to 2.45 to allow compatibility with python 3.13
Added filter for new sqlite3 “unclosed database” warnings in Python 3.13.

New Python 3.13 warnings see here : Added a filter to silence unclosed database warnings from sqlite3 introduced in Python 3.13. A comprehensive solution could be changing the code to get rid of these.

Are these changes tested?

Ran tests locally with 3.13.

Are there any user-facing changes?

Yes, ray version change required on Python 3.13.

@gregunz
Copy link
Author

gregunz commented Sep 30, 2025

image

@Fokko
Copy link
Contributor

Fokko commented Oct 20, 2025

@gregunz Now 3.9 has been dropped, are you interested in reviving this PR?

@gregunz
Copy link
Author

gregunz commented Oct 29, 2025

@Fokko: Sure, I can take the time this week to review it!

Fokko pushed a commit that referenced this pull request Jan 5, 2026
<!--
Thanks for opening a pull request!
-->

<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
<!-- Closes #${GITHUB_ISSUE_ID} -->

# Rationale for this change
Closes #1372, #2530

Based on #1377, #2536, #2658

Adds Python 3.13 throughout the repo
Upgrade `ray` to a version built with python 3.13
Add `filterwarnings` for `ResourceWarnings` coming from `sqlite` and
`ray` for now, will follow up with a proper fix

## Are these changes tested?
Yes
```
PYTHON=3.13 make install
PYTHON=3.13 make test
PYTHON=3.13 make test-integration
```

## Are there any user-facing changes?

<!-- In the case of user-facing changes, please add the changelog label.
-->

---------

Co-authored-by: Saurabh Kumar <[email protected]>
Co-authored-by: Saurabh Kumar <[email protected]>
@Fokko
Copy link
Contributor

Fokko commented Jan 5, 2026

Superseded by #2863

@Fokko Fokko closed this Jan 5, 2026
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