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

Merge featomic and featomic-c-api #349

Merged
merged 2 commits into from
Nov 25, 2024
Merged

Conversation

Luthaf
Copy link
Member

@Luthaf Luthaf commented Nov 22, 2024

This will allow to create a single archive containing all the relevant sources, to be included in the sdist for the python package.

This is part of #281


📚 Documentation preview 📚: https://featomic--349.org.readthedocs.build/en/349/

⚙️ Download Python wheels for this pull-request (you can install these with pip)

@Luthaf Luthaf force-pushed the merge-c-api branch 8 times, most recently from 8a3fc11 to 7b12e3f Compare November 25, 2024 14:19
This allow to disable the chemfiles dependency by default for the rust crate.

This makes examples in C and C++ a bit more complex, but also a lot closer
to how someone would actually use the code from these languages.
@Luthaf Luthaf marked this pull request as ready for review November 25, 2024 15:16
@Luthaf Luthaf requested a review from PicoCentauri November 25, 2024 15:20
featomic/benches/lode-spherical-expansion.rs Show resolved Hide resolved
featomic/build.rs Outdated Show resolved Hide resolved
This will make it easier to distirbute a single source archive
containing everything. featomic-c-api is now the `c-api` module in
featomic, enabled by the `c-api` cargo feature.
@Luthaf Luthaf merged commit 937679e into metatensor:main Nov 25, 2024
26 checks passed
@Luthaf Luthaf deleted the merge-c-api branch November 25, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants