Skip to content

Commit dbcf6f0

Browse files
committed
Fix NullPointer bug in CeSymm
Transformations needed to be set after refinement.
1 parent 576c766 commit dbcf6f0

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

  • biojava-structure/src/main/java/org/biojava/nbio/structure/symmetry/internal

biojava-structure/src/main/java/org/biojava/nbio/structure/symmetry/internal/CeSymm.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import java.util.ArrayList;
44
import java.util.List;
5+
56
import javax.vecmath.Matrix4d;
67

78
import org.biojava.nbio.structure.Atom;
@@ -15,6 +16,7 @@
1516
import org.biojava.nbio.structure.align.multiple.MultipleAlignment;
1617
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentEnsemble;
1718
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentEnsembleImpl;
19+
import org.biojava.nbio.structure.align.multiple.util.CoreSuperimposer;
1820
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentScorer;
1921
import org.biojava.nbio.structure.align.util.AFPChainScorer;
2022
import org.biojava.nbio.structure.jama.Matrix;
@@ -413,6 +415,8 @@ public MultipleAlignment analyze(Atom[] atoms, CESymmParameters param)
413415

414416
if (refined){
415417
if (msa == null) msa = SymmetryTools.fromAFP(afpChain, ca1);
418+
CoreSuperimposer imposer = new CoreSuperimposer();
419+
imposer.superimpose(msa);
416420
MultipleAlignmentScorer.calculateScores(msa);
417421
msa.putScore("isRefined", 1.0);
418422

0 commit comments

Comments
 (0)