argilla-io / argilla

Argilla is a collaboration tool for AI engineers and domain experts to build high-quality datasets

https://docs.argilla.io

Apache License 2.0

4.04k stars 381 forks source link

Closed frascuchon closed 2 weeks ago

frascuchon commented 3 weeks ago

Description

Type of change

How Has This Been Tested

Checklist

I added relevant documentation
I followed the style guidelines of this project
I did a self-review of my code
I made corresponding changes to the documentation
I confirm My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

frascuchon commented 3 weeks ago

oulianov commented 3 weeks ago

The dependencies install pyarrow 17.0.0, but python 3.13 is only supported for pyarrow 18.0.0 (released 2 days ago) https://github.com/apache/arrow/issues/43519

oulianov commented 3 weeks ago

I ran pdm lock with the latest pdm version using your pyproject.toml file. The solver gave me the right version of pyarrow (18.0).

So my guess is you just need to run pdm lock again and tests will pass for 3.13 :)

Let me know how it goes.

frascuchon commented 3 weeks ago

Thanks for the tip @oulianov. The pdm lock will update other dependencies making the whole workflow fails.

I defined the 18.0.0 as a dev dependency and everything is working.

@burtenshaw I think this is ready to merge.