aider icon indicating copy to clipboard operation
aider copied to clipboard

chore(deps): Support Python 3.13 and drop 3.9

Open reneleonhardt opened this issue 8 months ago • 3 comments

Closes #3037

reneleonhardt avatar Apr 08 '25 16:04 reneleonhardt

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Apr 08 '25 16:04 CLAassistant

If the tests are green, please try yourself @paul-gauthier 😄

reneleonhardt avatar Apr 08 '25 16:04 reneleonhardt

FYI there were many llama-index-core releases since you pinned 0.12.26 3 weeks ago, maybe test_help will work again 🤞😅

reneleonhardt avatar Apr 18 '25 12:04 reneleonhardt

@claui How is the review progressing?

reneleonhardt avatar May 05 '25 08:05 reneleonhardt

@reneleonhardt Sorry but I’m not actively reviewing this PR. The intent of my comment was to make sure you’re aware of the existing discussion. If GitHub shows me as a reviewer for this PR, then that’s misleading. Sorry about that.

In fact, I’ve filed #3516 (motivated by the Intel Mac requirement) back in March, i.e. before you filed this competing PR. I’m still waiting on @paul-gauthier’s feedback on it.

claui avatar May 05 '25 09:05 claui

Yeah, it's always confusing to me that a contributor isn't allowed to request a review 😞 You're right, it took me a long time to work around all those requirement lockfiles, real dependency management with uv or poetry would be so much easier 👍

reneleonhardt avatar May 05 '25 10:05 reneleonhardt

So... AI help for faster contributions isn't wanted and human help for dropping Python 3.9 isn't wanted either, now my work will never get reviewed 👍

reneleonhardt avatar May 10 '25 05:05 reneleonhardt

@reneleonhardt Many thanks for this PR! I've been using your PR locally for several weeks, and it worked perfectly both in 3.12/3.13 envs – truly appreciate you sharing this.

It looks like Python 3.13 support might be getting integrated by the maintainer more gradually, in separate parts – perhaps that's why this PR has some conflicts now. That's a challenging situation, and I can imagine it might be disheartening.

Despite that, it really shows that any good contribution you share can make a real difference for someone – your work here has definitely made one for me. Hope this feedback is a positive note!

ei-grad avatar May 14 '25 15:05 ei-grad

Thank you for trying my PR, feels better than being ignored and eventually closed... I honestly don't know why this hasn't been merged yet, I heard only positive feedback so far. But at least the maintainers have 5 months to merge this... before Python 3.14 will be released 😉 Which by the way also worked with my PR, at least as far as the pytest coverage is concerned. If you want you can pull again, I rebased, resolved conflicts and updated a little more.

reneleonhardt avatar May 14 '25 18:05 reneleonhardt

@reneleonhardt, could you resolve conflicts in this branch?

Slach avatar Aug 30 '25 15:08 Slach

This good old pip-compile style dependency management nobody is using anymore for years is an absolute nightmare. I could only spend hours again if my work would be constructively reviewed in a timely fashion and then merged if there are no problems. I have no trust in the maintainers anymore after adding the priority label and then keeping me waiting for 5 months, with 184 open PRs and refusing to allow AI review help which would save valuable contributors time.

Updates are literally time-sensitive, now every dependency constraint in every "requirement" file has to be reevaluated to satisfy all supported Python versions (supposedly 3.10, 3.11, 3.12 and maybe 3.13 if my work would've been merged).

It would be much easier to throw all ancient requirement files away and support the packaging standard (groups in pyproject.toml): https://packaging.python.org/en/latest/specifications/dependency-groups/ https://peps.python.org/pep-0735/ https://github.com/Aider-AI/aider/blob/main/scripts/pip-compile.sh

reneleonhardt avatar Aug 30 '25 15:08 reneleonhardt