-
Notifications
You must be signed in to change notification settings - Fork 89
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
Latticebond #62
Latticebond #62
Conversation
Note: The old expressions didn't add up: matrice sizes for the first PK was a mismatch of voigt and matrix form. This means that there was no way the old code was working correctly. With the fixed size matrices, this became a compile error, and I tried to remedy it in a way that made sense.
…y are all defined). Also remove unnecessary dtor.
Codecov Report
@@ Coverage Diff @@
## master #62 +/- ##
==========================================
+ Coverage 39.78% 40.1% +0.32%
==========================================
Files 1241 1241
Lines 125299 125260 -39
==========================================
+ Hits 49849 50235 +386
+ Misses 75450 75025 -425
Continue to review full report at Codecov.
|
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.
LGTM.
Though, I think there are some other improvements that could be done, this step is still in the right direction.
This addition should only affect the lattice models.
I have