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

BioJava - update library #83

Closed
julie-sullivan opened this issue Sep 12, 2012 · 6 comments
Closed

BioJava - update library #83

julie-sullivan opened this issue Sep 12, 2012 · 6 comments
Assignees
Milestone

Comments

@julie-sullivan
Copy link
Member

We are on a very old (7 years?) version of the library. Specifically, I think they fixed a memory leak.

@ghost ghost assigned boboppie Sep 12, 2012
@julie-sullivan
Copy link
Member Author

The current version is 3.0.4 but the legacy version is on 1.8.

The main reason for upgrading is that the FASTA doesn't process the amino acid Pyrrolysine. I think this was added in 2008 or so, after the biojava library we're using was written anyway.

http://en.wikipedia.org/wiki/FASTA_format

But if we are making this much effort to upgrade, we might as well upgrade to 3.0?

@ghost ghost assigned julie-sullivan Dec 4, 2012
@julie-sullivan
Copy link
Member Author

Fixed by 65589de, 7ee57ec, 1907ac0, ab1ee4f, 480edd8, d5f8e02

@ghost ghost assigned sergiocontrino Dec 2, 2013
@sergiocontrino
Copy link
Member

i think this is fixed

@julie-sullivan
Copy link
Member Author

@rachellyne I've removed PDB from FlyMine temporarily until we can update the BioJava library.

This is the error I get:

[integrate] WARNING: unknown group name MSE
java.lang.OutOfMemoryError: Requested array size exceeds VM limit
Dumping heap to /tmp/java_pid19540.hprof ...
Heap dump file created [3684876297 bytes in 15.089 secs]

BUILD FAILED
/local-homes/julie/git/intermine/imbuild/integrate.xml:54: The following error occurred while executing this line:
/local-homes/julie/git/intermine/imbuild/source.xml:162: java.lang.OutOfMemoryError: Requested array size exceeds VM limit
    at java.util.Arrays.copyOf(Arrays.java:2367)
    at java.lang.AbstractStringBuilder.expandCapacity(AbstractStringBuilder.java:130)
    at java.lang.AbstractStringBuilder.ensureCapacityInternal(AbstractStringBuilder.java:114)
    at java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:415)
    at java.lang.StringBuffer.append(StringBuffer.java:237)
    at org.biojava.bio.structure.PDBHeader.printTitle(PDBHeader.java:249)
    at org.biojava.bio.structure.PDBHeader.toPDB(PDBHeader.java:105)
    at org.biojava.bio.structure.io.FileConvert.toPDB(FileConvert.java:196)
    at org.biojava.bio.structure.StructureImpl.toPDB(StructureImpl.java:559)
    at org.intermine.bio.dataconversion.PdbConverter.processPDBFile(PdbConverter.java:162)
    at org.intermine.bio.dataconversion.PdbConverter.process(PdbConverter.java:104)
    at org.intermine.task.DirectoryConverterTask.execute(DirectoryConverterTask.java:104)
    at sun.reflect.GeneratedMethodAccessor4.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl

This happens in Java 6 (open-jdk only), 7 and 8. FlyMine is using sun's Java 6 where this works.

I suppose BioJava is creating an array that is too big.

@julie-sullivan
Copy link
Member Author

@sergiocontrino @rachellyne This is still a problem.

@julie-sullivan
Copy link
Member Author

PDB won't be included in this next release of FlyMine.

julie-sullivan pushed a commit that referenced this issue Jul 30, 2015
@julie-sullivan julie-sullivan mentioned this issue Jul 24, 2018
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants