Conversation
📝 WalkthroughWalkthroughThis 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 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
7f0ca05 to
a106892
Compare
There was a problem hiding this comment.
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.
|
The check |
a6c9b80 to
7db4bfb
Compare
|
@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) |
Summary by CodeRabbit
Chores
CI / Workflows
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.