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

Fix to concat0 #553

Merged
merged 3 commits into from
Oct 15, 2024
Merged

Fix to concat0 #553

merged 3 commits into from
Oct 15, 2024

Conversation

EricaCMitchell
Copy link
Collaborator

This PR addresses Issue #551

Original code from commit c7e2673 allowed nullptr for buf0 and buf1.

Fixes construction of unique_ptr and ensures buffer size bufsz is a multiple of alignment size_of(void*).

Was able to run mpirun -n 6 moldft input successfully on my machine with a total cpu time of 321.7s.

@evaleev
Copy link
Contributor

evaleev commented Oct 15, 2024

@fbischoff please try

@fbischoff
Copy link
Contributor

fbischoff commented Oct 15, 2024 via email

@evaleev
Copy link
Contributor

evaleev commented Oct 15, 2024

@fbischoff can an you please share input/metadata?

@fbischoff
Copy link
Contributor

fbischoff commented Oct 15, 2024 via email

@evaleev
Copy link
Contributor

evaleev commented Oct 15, 2024

@fbischoff runs fine for both of us. perhaps build issues on your side? can you please try e740317 again

@evaleev evaleev merged commit f76b84a into m-a-d-n-e-s-s:master Oct 15, 2024
12 checks passed
@EricaCMitchell EricaCMitchell deleted the pr_issue_551 branch October 22, 2024 21:18
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