Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace poetry with uv #360

Closed
ElliottKasoar opened this issue Nov 29, 2024 Discussed in #351 · 0 comments · Fixed by #358
Closed

Replace poetry with uv #360

ElliottKasoar opened this issue Nov 29, 2024 Discussed in #351 · 0 comments · Fixed by #358
Assignees
Labels
enhancement New/improved feature or request testing Unit testing or CI

Comments

@ElliottKasoar
Copy link
Member

Discussed in #351

Originally posted by ElliottKasoar November 15, 2024
As we increase the number of dependencies, I'm increasingly questioning the suitability of poetry for our dependency management.

In particular, the performance for resolving dependencies is becoming quite slow e.g. 1-5 minutes, and we're limited by the flexibility of optional dependencies, as I believe that poetry requires consistency between all extras, even if they're not installed, which may prevent us supporting/upgrading further MLIPs.

It's also worth noting that poetry 2.0 is on its way, with full PEP 621 support, which may include other improvements, but also means we may need to restructure pyproject.toml anyway.

In terms of improving efficiency, uv seems like a promising potential alternative, but I'm not sure whether it is sufficiently flexible or mature.

Any other suggestions are welcome.

@ElliottKasoar ElliottKasoar added enhancement New/improved feature or request testing Unit testing or CI labels Nov 29, 2024
@ElliottKasoar ElliottKasoar self-assigned this Nov 29, 2024
@ElliottKasoar ElliottKasoar linked a pull request Nov 29, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New/improved feature or request testing Unit testing or CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant