Skip to content

Commit 488d071

Browse files
committed
Organize MultipleAlignment code in a util package
1 parent ed95b34 commit 488d071

23 files changed

Lines changed: 292 additions & 232 deletions

biojava-structure-gui/src/main/java/org/biojava/nbio/structure/align/gui/MultipleAlignmentDisplay.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@
2626
import org.biojava.nbio.structure.align.gui.jmol.MultipleAlignmentJmol;
2727
import org.biojava.nbio.structure.align.multiple.Block;
2828
import org.biojava.nbio.structure.align.multiple.MultipleAlignment;
29-
import org.biojava.nbio.structure.align.multiple.MultipleSuperimposer;
30-
import org.biojava.nbio.structure.align.multiple.ReferenceSuperimposer;
29+
import org.biojava.nbio.structure.align.multiple.util.MultipleSuperimposer;
30+
import org.biojava.nbio.structure.align.multiple.util.ReferenceSuperimposer;
3131
import org.slf4j.Logger;
3232
import org.slf4j.LoggerFactory;
3333

biojava-structure-gui/src/main/java/org/biojava/nbio/structure/align/gui/MySaveFileListener.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
import org.biojava.nbio.structure.Atom;
2424
import org.biojava.nbio.structure.align.model.AFPChain;
2525
import org.biojava.nbio.structure.align.multiple.MultipleAlignment;
26-
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentWriter;
26+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentWriter;
2727
import org.biojava.nbio.structure.align.util.AtomCache;
2828
import org.biojava.nbio.structure.align.util.UserConfiguration;
2929
import org.biojava.nbio.structure.align.webstart.WebStartMain;

biojava-structure-gui/src/main/java/org/biojava/nbio/structure/align/gui/aligpanel/MultipleAligPanel.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@
4444
import org.biojava.nbio.structure.align.model.AFPChain;
4545
import org.biojava.nbio.structure.align.multiple.MultipleAlignment;
4646
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentEnsembleImpl;
47-
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentTools;
48-
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentWriter;
47+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentTools;
48+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentWriter;
4949
import org.biojava.nbio.structure.align.util.AFPAlignmentDisplay;
5050
import org.biojava.nbio.structure.gui.events.AlignmentPositionListener;
5151
import org.biojava.nbio.structure.gui.util.AlignedPosition;

biojava-structure-gui/src/main/java/org/biojava/nbio/structure/align/gui/aligpanel/MultipleStatusDisplay.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929

3030
import org.biojava.nbio.structure.Atom;
3131
import org.biojava.nbio.structure.align.gui.jmol.JmolTools;
32-
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentTools;
32+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentTools;
3333
import org.biojava.nbio.structure.gui.events.AlignmentPositionListener;
3434
import org.biojava.nbio.structure.gui.util.AlignedPosition;
3535

biojava-structure-gui/src/main/java/org/biojava/nbio/structure/align/gui/jmol/MultipleAlignmentJmol.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
import org.biojava.nbio.structure.align.multiple.Block;
3333
import org.biojava.nbio.structure.align.multiple.BlockSet;
3434
import org.biojava.nbio.structure.align.multiple.MultipleAlignment;
35-
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentWriter;
35+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentWriter;
3636
import org.biojava.nbio.structure.align.webstart.AligUIManager;
3737
import org.biojava.nbio.structure.jama.Matrix;
3838
import org.jcolorbrewer.ColorBrewer;

biojava-structure/src/main/java/demo/DemoMultipleMC.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@
1010
import org.biojava.nbio.structure.StructureException;
1111
import org.biojava.nbio.structure.align.ce.CeCPMain;
1212
import org.biojava.nbio.structure.align.multiple.MultipleAlignment;
13-
import org.biojava.nbio.structure.align.multiple.MultipleAlignmentWriter;
1413
import org.biojava.nbio.structure.align.multiple.mc.MultipleMcMain;
1514
import org.biojava.nbio.structure.align.multiple.mc.MultipleMcParameters;
15+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentWriter;
1616
import org.biojava.nbio.structure.align.util.AtomCache;
1717

1818
/**

biojava-structure/src/main/java/org/biojava/nbio/structure/align/multiple/BlockImpl.java

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,11 @@
44
import java.util.ArrayList;
55
import java.util.List;
66

7+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentTools;
8+
79
/**
810
* General implementation of a {@link Block} that supports any type of
9-
* alignment with gaps.
11+
* sequential alignment with gaps.
1012
*
1113
* @author Aleix Lafita
1214
* @since 4.1.0
@@ -114,17 +116,6 @@ public int getCoreLength() {
114116
}
115117

116118
protected void updateCoreLength() {
117-
coreLength = 0;
118-
//Count how many positions do not have gaps in any structure
119-
for (int col=0; col<length(); col++){
120-
boolean core = true;
121-
for (int str=0; str<size(); str++){
122-
if (alignRes.get(str).get(col) == null){
123-
core = false;
124-
break;
125-
}
126-
}
127-
if (core) coreLength++;
128-
}
119+
coreLength = MultipleAlignmentTools.getCorePositions(this).size();
129120
}
130121
}

biojava-structure/src/main/java/org/biojava/nbio/structure/align/multiple/MultipleAlignmentEnsembleImpl.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
import org.biojava.nbio.structure.StructureTools;
1515
import org.biojava.nbio.structure.align.helper.AlignTools;
1616
import org.biojava.nbio.structure.align.model.AFPChain;
17+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentScorer;
1718
import org.biojava.nbio.structure.align.util.AtomCache;
1819
import org.biojava.nbio.structure.jama.Matrix;
1920

biojava-structure/src/main/java/org/biojava/nbio/structure/align/multiple/MultipleSuperimposer.java

Lines changed: 0 additions & 23 deletions
This file was deleted.

biojava-structure/src/main/java/org/biojava/nbio/structure/align/multiple/ScoresCache.java

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

33
import java.util.Set;
44

5+
import org.biojava.nbio.structure.align.multiple.util.MultipleAlignmentScorer;
6+
57
/**
68
* Interface for classes which implement a temporary cache for various numeric
79
* scores, e.g. RMSD, TM-Score, etc.

0 commit comments

Comments
 (0)