chore(build): migrate to pyproject

lightly-ai / lightly

A python library for self-supervised learning on images.

https://docs.lightly.ai/self-supervised-learning/

MIT License

2.83k stars 246 forks source link

chore(build): migrate to pyproject #1559

Closed SauravMaheshkar closed 1 week ago

SauravMaheshkar commented 2 weeks ago

Ref: #1085

Move requirement files to pyproject.toml
Updated GA to cache deps from pyproject.toml
Update python packaging using build
drop MANIFEST.in in favour of tool.setuptools in pyproject.toml
quick fix to lightly/models/modules/ijepa.py to silence black warnings

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.81%. Comparing base (8346959) to head (8b9fe66).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #1559 +/- ## ======================================= Coverage 81.81% 81.81% ======================================= Files 144 144 Lines 6119 6119 ======================================= Hits 5006 5006 Misses 1113 1113 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

philippmwirth commented 2 weeks ago

Hey @SauravMaheshkar thanks for the contribution! I will take a look ASAP.

philippmwirth commented 2 weeks ago

It looks like we will have to fix the numpy dependency problems first. Will take a look at this tomorrow.

philippmwirth commented 2 weeks ago

Hey @SauravMaheshkar , we decided to temporarily limit the numpy version to < 2.0 can you update the pull request accordingly please?

SauravMaheshkar commented 2 weeks ago

@philippmwirth fixed in 3f542be, CI looking green now 😄

SauravMaheshkar commented 1 week ago

@philippmwirth It might also be worth adding tox configuration to pyproject.toml viz. https://tox.wiki/en/stable/config.html#pyproject-toml