<div dir="ltr">Hi Steve,<div><br></div><div>I'd say these are excellent feature requests for the current mmCIF parsing framework. </div><div><br></div><div>We definitely want to enable all of the categories mentioned by you. The data categories need to be added in either the SimpleMmCifParser/Consumer, and the ChemCompConsumer. </div><div><br></div><div>If you want to provide a patch for parsing ligand information, sites, and ssbonds, that would be welcome. I'd say these should go all into the SimpleMMCIFConsumer.</div><div><br></div><div>Having said that, we need to double check that some of your requests are not already implemented. E.g. there is already the BondMaker class that assigns bonds based on mmCif definitions. (check the shouldCreateAtomBonds flag in FileParsingParameters)</div><div><br></div><div>Do you want to open up (multiple) tickets on GitHub for the requested data categories, then we can discuss them in detail there?</div><div><br></div><div>Thanks,</div><div><br></div><div>Andreas</div><div><div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 27, 2015 at 1:11 PM, Steve Darnell <span dir="ltr"><<a href="mailto:darnells@dnastar.com" target="_blank">darnells@dnastar.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Greetings,<br>
<br>
Our group uses the BioJava mmCIF parser to handle reading mmCIF files to provide structural information. The parsing and writing of mmCIF provide some of the important data entries but we would like to extend these to provide more details. We are looking in particular to add the ability to parse ligand bond information, the _struct_site records for sites, and _struct_conn for disulfide bonds. We are hoping to contribute any implementations for parsing back to the BioJava code base.<br>
<br>
For the ligand bond information, we could parse connectivity via the Chemical Component library since the bond information is not present within mmCIF as CONECT was for PDB.<br>
<br>
Is there planned development to get the equivalent bond information for ligands in the SimpleMMCIFParser as is present for CONECT records for PDB parsing? If we were to implement parsing of ligand information, sites, and ssbonds, would it be best to add this SimpleMMCIFConsumer or as another consumer?<br>
<br>
Is this discussion best suited for the github issue tracker? If so, we will start a discussion there.<br></blockquote></div><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr">-----------------------------------------------------------------------<br>Dr. Andreas Prlic<br>RCSB PDB Protein Data Bank</div><div>Technical & Scientific Team Lead</div><div dir="ltr">University of California, San Diego<div><br></div><div>Editor Software Section <br><div>PLOS Computational Biology<div><div><div><br></div><div>BioJava Project Lead<br>-----------------------------------------------------------------------<br></div></div></div></div></div></div></div></div></div>
</div></div></div>