Merge pull request #2191 from RosettaCommons/JWLabonte/sugars/database
Carbohydrates: A gift for @raemisch
All unit tests pass. Beautifier is complaining about code that isn't ugly. Integration test changes are all reporting on a new ResidueType.
Rocco Moretti 7 years Just a note that the beautification test is a perma-fail one. You either have to conform to the unrealistic standards of beauty that society (or rather the test) expects of you, or be the change you want to see in the world and fix the beautifier script. (Or lobby Andrew to fix it for you.)
Labonte 7 years Ah, I thought it was like an integration test. (I would strongly argue that it should be like an integration test, but that's for another time.)
In this case, then, the change was from an earlier merge. I had thought that that one was a different problem, but revisiting it, it was indeed a spacing issue. It should be fixed now. (I do not see what is wrong with Andy's code.)
notify author
notify list [rosetta-logs@googlegroups.com]