-
Notifications
You must be signed in to change notification settings - Fork 139
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
change product to hardcoded number fixes #1012 fixes #998 fixes #997 #1051
base: main
Are you sure you want to change the base?
change product to hardcoded number fixes #1012 fixes #998 fixes #997 #1051
Conversation
e77d458
to
a6aeb3f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If these arrays were allocated with the right dimensions there is no need at all to specify the shape.
…-lab#998 fixes grimme-lab#997 Signed-off-by: James H. Brown <james.h.brown@dartmouth.edu>
….harry.brown@gmail.com> Signed-off-by: James H. Brown <james.h.brown@dartmouth.edu>
Co-authored-by: Sebastian Ehlert <28669218+awvwgk@users.noreply.github.com> Signed-off-by: James H. Brown <james.h.brown@dartmouth.edu>
Co-authored-by: Sebastian Ehlert <28669218+awvwgk@users.noreply.github.com> Signed-off-by: James H. Brown <james.h.brown@dartmouth.edu>
f013021
to
c17e46f
Compare
I agree. Didn't know if you wanted them explicitly defined for some reason. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for sharing. Looks good to me now.
I had the same compilation error as reference in #1012 , #998, and #997.
This is clearly some sort of bug in gcc as it is clearly passing an array to product.
Unless I am misunderstanding something, changing to hardcoded numbers should not pose a problem. If desired, I could also do the explicit product. e.g.
(2*ewaldCutD(1) + 1)*(2*ewaldCutD(2) + 1)*(2*ewaldCutD(3) + 1)
Closes #1012
Closes #998
Closes #997