Skip to content
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

Error running on Alphafold models #113

Closed
sbliven opened this issue Jan 18, 2022 · 4 comments
Closed

Error running on Alphafold models #113

sbliven opened this issue Jan 18, 2022 · 4 comments
Assignees
Milestone

Comments

@sbliven
Copy link
Collaborator

sbliven commented Jan 18, 2022

Running CE-Symm on alphafold models give an error:

$ runCESymm.sh -J AF-A0A024R1R8-F1-model_v2.cif
Structure       NumRepeats      SymmGroup       Reason
594 [pool-2-thread-1] ERROR workers.CeSymmWorker - Could not complete job: AF-A0A024R1R8-F1-model_v2.cif
java.lang.IllegalArgumentException: ID can not be null
        at org.biojava.nbio.structure.PdbId.<init>(PdbId.java:78) ~[cesymm-2.2.1.jar:2.2.1]
        at org.biojava.nbio.structure.URLIdentifier.toCanonical(URLIdentifier.java:125) ~[cesymm-2.2.1.jar:2.2.1]
        at org.biojava.nbio.structure.URLIdentifier.reduce(URLIdentifier.java:130) ~[cesymm-2.2.1.jar:2.2.1]
        at org.biojava.nbio.structure.align.client.StructureName.reduce(StructureName.java:511) ~[cesymm-2.2.1.jar:2.2.1]
        at org.biojava.nbio.structure.align.util.AtomCache.getStructure(AtomCache.java:504) ~[cesymm-2.2.1.jar:2.2.1]
        at workers.CeSymmWorker.run(CeSymmWorker.java:58) [cesymm-2.2.1.jar:2.2.1]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130) [?:?]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630) [?:?]
        at java.lang.Thread.run(Thread.java:832) [?:?]

The error occurs in both PDB and CIF files

@sbliven sbliven self-assigned this Jan 18, 2022
@josemduarte
Copy link
Member

This looks like a consequence of the new PdbId class handling, introduced in biojava/biojava#950

In principle the parser should not try to set a PDB id for AlphaFold case. But it looks like it is trying that. Do you know if you are parsing PDB format or mmCIF format?

@youkha
Copy link

youkha commented Jan 18, 2022 via email

@josemduarte
Copy link
Member

@sbliven this is fixed in biojava 6.0.5

@sbliven
Copy link
Collaborator Author

sbliven commented Feb 2, 2022

Thanks @josemduarte

@sbliven sbliven closed this as completed Feb 2, 2022
sbliven added a commit to sbliven/symmetry that referenced this issue Feb 2, 2022
@sbliven sbliven added this to the CE-Symm 2.2 milestone Feb 2, 2022
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

No branches or pull requests

3 participants