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

support AtomsBase v0.4 #25

Merged
merged 2 commits into from
Sep 3, 2024
Merged

support AtomsBase v0.4 #25

merged 2 commits into from
Sep 3, 2024

Conversation

tjjarvinen
Copy link
Collaborator

No description provided.

@cortner
Copy link
Member

cortner commented Aug 27, 2024

thank you. Doesn't this need updates to AtomsCalculatorsUtilities first though? In particular issue ACU issue # 9

@tjjarvinen
Copy link
Collaborator Author

tjjarvinen commented Aug 27, 2024

The tests need the new version to run properly. It does pass on my computer with correct versions though

@cortner
Copy link
Member

cortner commented Aug 28, 2024

Well, they will pass here as well because they will just install the old ACU which restricts AB to < 0.4. So I prefer to first update ACU to be AB 0.4 compatible (even if it means a breaking change) and then come back to this PR?

@tjjarvinen
Copy link
Collaborator Author

Yes I am doing that atm. ACU will be tagged new release once the CI passes. Then I will come to this.

There is also a bug with Lennard-Jones potential, so I try to get in a fix for that too.

@cortner
Copy link
Member

cortner commented Aug 28, 2024

great, thank you.

@tjjarvinen
Copy link
Collaborator Author

AtomsBuilder needs an update too, to get the tests run on ABv04

@cortner
Copy link
Member

cortner commented Aug 28, 2024

AtomsBuilder needs an update too

about to be merged into General

@cortner cortner merged commit a98a45c into main Sep 3, 2024
4 checks passed
@cortner cortner mentioned this pull request Sep 3, 2024
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