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

Update metatensor #246

Merged
merged 2 commits into from
Oct 11, 2023
Merged

Update metatensor #246

merged 2 commits into from
Oct 11, 2023

Conversation

Luthaf
Copy link
Member

@Luthaf Luthaf commented Oct 10, 2023

@github-actions
Copy link

github-actions bot commented Oct 10, 2023

Here is a pre-built version of the code in this pull request: wheels.zip, you can install it locally by unzipping wheels.zip and using pip to install the file matching your system

@Luthaf Luthaf force-pushed the update-metatensor branch from 616dcc7 to 0a2fd7d Compare October 11, 2023 10:09
@Luthaf Luthaf requested a review from PicoCentauri October 11, 2023 10:09
I used clang-tidy for C++ code, and clippy for Rust code
@Luthaf Luthaf force-pushed the update-metatensor branch from 0a2fd7d to c48754d Compare October 11, 2023 10:19
Copy link
Member

@jwa7 jwa7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and CI passed - shall we merge @PicoCentauri ?

@PicoCentauri PicoCentauri merged commit 434f923 into master Oct 11, 2023
24 checks passed
@PicoCentauri PicoCentauri deleted the update-metatensor branch October 11, 2023 11:16
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.

3 participants