Skip to content

Set coefficients to 0 if they were missing from input file, this is a…#349

Open
JoshuaSBrown wants to merge 1 commit intoshirtsgroup:masterfrom
JoshuaSBrown:master
Open

Set coefficients to 0 if they were missing from input file, this is a…#349
JoshuaSBrown wants to merge 1 commit intoshirtsgroup:masterfrom
JoshuaSBrown:master

Conversation

@JoshuaSBrown
Copy link
Copy Markdown
Contributor

… result of addressing error where dihedral angle expected five coefficients but only four were provided. This is to address this issue:

#348

… result of addressing error where dihedral angle expected five coefficients but only four were provided
@JoshuaSBrown
Copy link
Copy Markdown
Contributor Author

Is anyone going to merge this?

@mrshirts
Copy link
Copy Markdown
Contributor

Might it be possible to add at the same time a simple test that fails with the current code, but succeeds with the new code? Look in intermol/tests for examples of minimal tests.

@JoshuaSBrown
Copy link
Copy Markdown
Contributor Author

Ok, I'll see what I can do.

@orionarcher
Copy link
Copy Markdown

Any plans to merge this? I am encountering the same error.

orionarcher added a commit to orionarcher/InterMol that referenced this pull request Oct 15, 2021
@orionarcher orionarcher mentioned this pull request Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants