Skip to content

Remove old mmCIF parsing and refactor chemcomp handling #912

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.

Sign up for GitHub

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

Merged
josemduarte merged 27 commits into from
Feb 26, 2021
Merged

Remove old mmCIF parsing and refactor chemcomp handling #912

Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
e3bad37
detach mmcif.model.DatabasePdbrevRecord
sbittrich Nov 18, 2020
c290ea9
wip (most of structure parsing done)
sbittrich Jan 13, 2021
993665a
wip (TODO chemcomp parsing)
sbittrich Jan 14, 2021
1b164a9
compiling
sbittrich Jan 20, 2021
b992e74
metal parsing
sbittrich Jan 25, 2021
61bd8e8
new filetype param
sbittrich Jan 25, 2021
e7f563d
fix bcif download
sbittrich Jan 25, 2021
0c00cac
fix chemcomp type
sbittrich Jan 25, 2021
16792bd
Merge remote-tracking branch 'upstream/master' into bcif-integration
sbittrich Jan 26, 2021
85b6c6b
chem comp parsing errors & missing crystal translation
sbittrich Jan 26, 2021
e9c59f6
fix entity parsing
sbittrich Jan 26, 2021
28815db
handle missing rfree/rwork
sbittrich Jan 26, 2021
f36f82c
SCOP hack to force loading of MMTF
sbittrich Jan 26, 2021
63d2ee7
bcif/models URL is a constant
sbittrich Jan 26, 2021
2d4cc74
StructureInterface#toMMCIF
sbittrich Jan 26, 2021
b7a40c5
TODO TestHardBioUnits
sbittrich Jan 27, 2021
2f91dbf
those aren't the FloatColumns you're looking for
sbittrich Jan 27, 2021
9f42c9c
trigger build
sbittrich Jan 27, 2021
454830a
docs
sbittrich Jan 27, 2021
4f44fce
Group.java from upstream
sbittrich Feb 1, 2021
53982b0
update CHANGELOG.md
sbittrich Feb 2, 2021
19766ca
update AtomCacheTest
sbittrich Feb 3, 2021
58efc46
Merge remote-tracking branch 'upstream/master' into bcif-integration
sbittrich Feb 4, 2021
e94f047
increase QCP precision threshold (try to fix #914)
sbittrich Feb 10, 2021
83cf763
Revert "increase QCP precision threshold (try to fix #914)"
sbittrich Feb 12, 2021
54f4c2b
dedicated test for SuperPositionQCP issues (#914)
sbittrich Feb 24, 2021
288b18c
switch from QCP to SVD
sbittrich Feb 24, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
handle missing rfree/rwork
  • Loading branch information
@sbittrich
sbittrich committed Jan 26, 2021
commit 28815dbd5c5d08cdad8a7b2e486cefb3b7a02106
5 changes: 3 additions & 2 deletions biojava-structure/src/main/java/org/biojava/nbio/structure/io/cif/StructureConsumerImpl.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
import org.rcsb.cif.model.FloatColumn;
import org.rcsb.cif.model.IntColumn;
import org.rcsb.cif.model.StrColumn;
import org.rcsb.cif.model.ValueKind;
import org.rcsb.cif.schema.mm.AtomSite;
import org.rcsb.cif.schema.mm.AtomSites;
import org.rcsb.cif.schema.mm.AuditAuthor;
Expand Down Expand Up @@ -829,7 +830,7 @@ public void consumeRefine(Refine refine) {
logger.warn("More than 1 Rfree value present, will use last one {} and discard previous {}",
lsRFactorRFree, String.format("%4.2f",pdbHeader.getRfree()));
}
if (lsRFactorRFree.isDefined()) {
if (lsRFactorRFree.isDefined() && lsRFactorRFree.getValueKind(rowIndex) == ValueKind.PRESENT) {
pdbHeader.setRfree((float) lsRFactorRFree.get(rowIndex));
} else {
// some entries like 2ifo haven't got this field at all
Expand All @@ -842,7 +843,7 @@ public void consumeRefine(Refine refine) {
logger.warn("More than 1 R work value present, will use last one {} and discard previous {} ",
lsRFactorRWork, String.format("%4.2f",pdbHeader.getRwork()));
}
if (lsRFactorRWork.isDefined()) {
if (lsRFactorRWork.isDefined() && lsRFactorRWork.getValueKind(rowIndex) == ValueKind.PRESENT) {
pdbHeader.setRwork((float) lsRFactorRWork.get(rowIndex));
} else {
logger.info("_refine.ls_R_factor_R_work not present, not parsing R-work value");
Expand Down