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

Zmat params #780

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Zmat params #780

wants to merge 3 commits into from

Conversation

alongd
Copy link
Member

@alongd alongd commented Dec 30, 2024

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.

@alongd alongd requested a review from LeenFahoum December 30, 2024 12:33
LeenFahoum
LeenFahoum previously approved these changes Dec 30, 2024
Copy link

codecov bot commented Dec 30, 2024

Codecov Report

Attention: Patch coverage is 79.31034% with 36 lines in your changes missing coverage. Please review.

Project coverage is 74.09%. Comparing base (35dcafa) to head (c3ea0ba).

Files with missing lines Patch % Lines
arc/species/converter.py 75.51% 29 Missing and 7 partials ⚠️
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     
Flag Coverage Δ
unittests 74.09% <79.31%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

Variable sphere_error is not used.
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

Variable angle_error is not used.

sphere_error = sphere_eq(*coord)
angle_error = angle_eq(*coord)
dihedral_error = dihedral_eq(*coord)

Check notice

Code scanning / CodeQL

Unused local variable Note

Variable dihedral_error is not used.
alongd added 3 commits January 2, 2025 22:44
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants