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 README.md Install Steps #198

Merged
merged 3 commits into from
Oct 8, 2024
Merged

Update README.md Install Steps #198

merged 3 commits into from
Oct 8, 2024

Conversation

mikemhenry
Copy link
Contributor

Fixes #195

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@13af90b). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files

@mattwthompson
Copy link
Contributor

Purely out of curiosity, how far away is Espaloma from running natively on Apple Silicon? I assume it's a few of the dependencies

@mikemhenry
Copy link
Contributor Author

We did get DGL cross compiled, so other deps might need work but soon ™️

Copy link
Contributor

@ijpulidos ijpulidos left a comment

Choose a reason for hiding this comment

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

LGTM!

@mikemhenry
Copy link
Contributor Author

mikemhenry commented Oct 7, 2024

Might need to drop the osx subdir trick if dgl 2 works ref #220

@mikemhenry
Copy link
Contributor Author

This is true today, but will need to be update with the next espaloma release

@mikemhenry mikemhenry merged commit 907d183 into main Oct 8, 2024
6 of 7 checks passed
@mikemhenry mikemhenry deleted the mikemhenry-patch-3 branch October 8, 2024 19: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.

Issue with conda installation
4 participants