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
Fixed issues around reduced format #41
Conversation
declared List<Entity> entities transient to avoid serializing temporary data structures, added setMmtfProducer method.
comprehensive test for the reduced data structure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I've commented on a couple of issues. Specially it'd be good to have testing on other edge cases.
else if (chainType.equals("non-polymer")){ | ||
for(int i=0; i<structureDataInterface.getNumAtomsInGroup(groupType); i++){ | ||
outList.add(i); | ||
} else if (! structure.getGroupName(groupIndex).equals("HOH")){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about deuterated water?
public void test() throws IOException, ParseException { | ||
StructureDataInterface full = getDefaultFullData(); | ||
StructureDataInterface reduced = ReducedEncoder.getReduced(getDefaultFullData()); | ||
compareFullVsReduced(full.getStructureId(), full, reduced); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about including a few more cases here, not only 4cup. E.g. structure with nucleotides, structure with water marked as non-polymer, structure with alt conformations, etc...
DOD as a solvent. Now also checking for DOD in the ReducedEncoder.
Thanks for the changes @pwrose ! One issue now, could you check why the tests are not passing? By the way, I forgot to mention yesterday that a possible way to do the tests is by using https://github.com/rcsb/mmtf/tree/master/test-suite |
Btw, I have used the test-suite files. I've put a local copy of the files
in the resources folder so the tests can be run offline.
…On Tue, Jun 13, 2017 at 8:36 AM, Jose Manuel Duarte < ***@***.***> wrote:
Thanks for the changes @pwrose <https://github.com/pwrose> ! One issue
now, could you check why the tests are not passing?
By the way, I forgot to mention yesterday that a possible way to do the
tests is by using https://github.com/rcsb/mmtf/tree/master/test-suite
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#41 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADuwEO5fm40T_6wDXm8UHcbwSL27a5mwks5sDqxqgaJpZM4N331R>
.
|
No description provided.