Skip to content

Bug fix and update ChunkSplitters#22

Merged
cortner merged 4 commits intoACEsuit:mainfrom
tjjarvinen:chunks_v3
Apr 17, 2025
Merged

Bug fix and update ChunkSplitters#22
cortner merged 4 commits intoACEsuit:mainfrom
tjjarvinen:chunks_v3

Conversation

@tjjarvinen
Copy link
Copy Markdown
Contributor

This update needed for to update ACEpotentials to AtomsBase v0.5 and to fix ACEsuit/ACEpotentials.jl#283

Update to ChunkSplitters v3 and fix an assosiated bug

#This works
domain = 1:10  # = 1:10

#This does not
domain = 2:10  # = 1:9

I also added version bounds for the deps. These will be vital to keep things under control in the future.

And Finally, I added version bump, so only merge and registration needed.

@cortner cortner merged commit 1869130 into ACEsuit:main Apr 17, 2025
2 checks passed
@cortner
Copy link
Copy Markdown
Member

cortner commented Apr 17, 2025

thank you

@tjjarvinen tjjarvinen deleted the chunks_v3 branch April 17, 2025 19:59
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.

Regarding issue in running simulation using ACE-potential in Molly.jl package

2 participants