-
Notifications
You must be signed in to change notification settings - Fork 2
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
Initial addition of angle constraints #8
base: main
Are you sure you want to change the base?
Conversation
@rsdefever This is on my todo list. I might not get to it for a week. |
e3109c0
to
5cb41a8
Compare
The angle constraints now seem to be working in a simple case (an important simple case!) of a water molecule (so two bond constraints with one angle constraint). A more difficult test case of a dimethyl ether molecule still reports the same error on the solver. When I check the geometry, all the bonds meet the bond-length constraints. However, there are several angles that do not meet the angle constraints. I am unsure if it is feasible for all the angles to be satisfied, although the total number of constraints is 21 and the number of variables is 27. Here is the molecule: |
/azp run |
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
@adowling2 can you have a look at my attempted implementation of angle constraints. I'm getting the following error, even for a very simple test case. Any ideas?