Skip to content

feat: remove python 3.9#65

Merged
m1so merged 4 commits intomainfrom
fpyrek/drop-python-3-9
Jan 25, 2026
Merged

feat: remove python 3.9#65
m1so merged 4 commits intomainfrom
fpyrek/drop-python-3-9

Conversation

@FilipPyrek
Copy link
Contributor

@FilipPyrek FilipPyrek commented Jan 20, 2026

Summary by CodeRabbit

  • Chores

    • Updated default Python baseline from 3.9 to 3.10.
    • Added support for Python 3.11, 3.12, and 3.13.
    • Removed Python 3.9 support and related legacy image/build artifacts.
  • CI / Workflows

    • Adjusted workflows and build matrices to use 3.10 as the IR baseline and include the expanded Python matrix.
    • Increased CI step timeouts for several long-running build and push jobs.
  • Documentation

    • Updated supported tags and usage notes to reflect the new Python versions.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

This pull request removes Python 3.9 support and sets Python 3.10 as the baseline across the repository. CI (.circleci/config.yml) timeouts for long-running steps were increased and Python version matrices/defaults updated to 3.10–3.13. Dockerfiles (ir/ir-base and python/conda) now use ARG PYTHON_VERSION=3.10. The file python/python/Dockerfile.python3.9 was removed. Documentation in python/readme.md and supported tags were updated to reflect the new Python versions.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly describes the main change: removing Python 3.9 support across Dockerfiles, CI config, and documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@FilipPyrek FilipPyrek force-pushed the fpyrek/drop-python-3-9 branch from 7f0ca05 to a106892 Compare January 22, 2026 15:53
@deepnote deepnote deleted a comment from linear bot Jan 22, 2026
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@python/readme.md`:
- Around line 5-6: The Markdown heading "Supported Tags and Respective
Dockerfile Links" lacks a blank line after it; update the section by inserting a
single blank line immediately after the heading (i.e., after "# Supported Tags
and Respective Dockerfile Links") so the following list line with the tags
renders correctly and satisfies markdownlint.
- Around line 6-7: The referenced constraint files (URLs
https://tk.deepnote.com/constraints3.10.txt through constraints3.13.txt) are
missing and breaking builds; either deploy the missing constraint files at those
exact URLs or remove the --constraint references and README links that point to
them. Update python/readme.md to stop linking the nonexistent constraint URLs
and edit the Dockerfiles that reference these constraints (e.g., the Dockerfiles
in python/python and python/datascience/Dockerfile.datascience) to either (a)
remove the pip --constraint <url> flags or (b) replace them with valid hosted
constraint files; ensure the Dockerfile pip install lines and README entries
consistently reflect the chosen fix.

@FilipPyrek
Copy link
Contributor Author

FilipPyrek commented Jan 23, 2026

The check R with libs 4.4.3 has been running for 30 mins already and it's still building and logging. 🤔 and it times out

@FilipPyrek FilipPyrek force-pushed the fpyrek/drop-python-3-9 branch from a6c9b80 to 7db4bfb Compare January 23, 2026 16:22
@m1so
Copy link
Contributor

m1so commented Jan 25, 2026

@FilipPyrek long build times were caused by not compiling R packages with using multiple cores (see: #67)

pre-compiled packages could be used as well, but we'd need to update from Debian Bullseye to Bookworm (https://wiki.debian.org/DebianReleases)

@m1so m1so merged commit 8af9a5b into main Jan 25, 2026
23 checks passed
@m1so m1so deleted the fpyrek/drop-python-3-9 branch January 25, 2026 12:31
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.

3 participants