-
Notifications
You must be signed in to change notification settings - Fork 23
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
Zmat params #780
base: main
Are you sure you want to change the base?
Zmat params #780
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #780 +/- ##
==========================================
+ Coverage 74.06% 74.09% +0.03%
==========================================
Files 101 101
Lines 28054 28227 +173
Branches 5872 5883 +11
==========================================
+ Hits 20777 20916 +139
- Misses 5803 5831 +28
- Partials 1474 1480 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
b180673
to
cb8dfc3
Compare
angle_constraint_ = angle_eq(x, y, z) | ||
dihedral_constraint_ = dihedral_eq(x, y, z) | ||
|
||
sphere_error = sphere_eq(*coord) |
Check notice
Code scanning / CodeQL
Unused local variable Note
dihedral_constraint_ = dihedral_eq(x, y, z) | ||
|
||
sphere_error = sphere_eq(*coord) | ||
angle_error = angle_eq(*coord) |
Check notice
Code scanning / CodeQL
Unused local variable Note
|
||
sphere_error = sphere_eq(*coord) | ||
angle_error = angle_eq(*coord) | ||
dihedral_error = dihedral_eq(*coord) |
Check notice
Code scanning / CodeQL
Unused local variable Note
f62fbd1
to
5e91bd3
Compare
To add a new atom into an existing XYZ coords using r, a, and d internal coords defined independently
added test for add_atom_to_xyz_using_internal_coords
Allow users to add an atom to a ZMat even if it is defined with respect to other inconsistent atoms, e.g.: r or atom R-X, a of atoms A-B-X, and d of atoms N-L-M-X.
A test was added.