git://source.jalview.org
/
jalview.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
JAL-4089 use column selection to superpose if 4 or more selected
[jalview.git]
/
src
/
jalview
/
structures
/
models
/
AAStructureBindingModel.java
diff --git
a/src/jalview/structures/models/AAStructureBindingModel.java
b/src/jalview/structures/models/AAStructureBindingModel.java
index
44d698d
..
8436b48
100644
(file)
--- a/
src/jalview/structures/models/AAStructureBindingModel.java
+++ b/
src/jalview/structures/models/AAStructureBindingModel.java
@@
-20,8
+20,6
@@
*/
package jalview.structures.models;
*/
package jalview.structures.models;
-import java.util.Locale;
-
import java.awt.Color;
import java.io.File;
import java.io.IOException;
import java.awt.Color;
import java.io.File;
import java.io.IOException;
@@
-31,6
+29,7
@@
import java.util.BitSet;
import java.util.HashMap;
import java.util.LinkedHashMap;
import java.util.List;
import java.util.HashMap;
import java.util.LinkedHashMap;
import java.util.List;
+import java.util.Locale;
import java.util.Map;
import javax.swing.SwingUtilities;
import java.util.Map;
import javax.swing.SwingUtilities;
@@
-43,6
+42,7
@@
import jalview.api.StructureSelectionManagerProvider;
import jalview.api.structures.JalviewStructureDisplayI;
import jalview.bin.Console;
import jalview.datamodel.AlignmentI;
import jalview.api.structures.JalviewStructureDisplayI;
import jalview.bin.Console;
import jalview.datamodel.AlignmentI;
+import jalview.datamodel.ColumnSelection;
import jalview.datamodel.HiddenColumns;
import jalview.datamodel.MappedFeatures;
import jalview.datamodel.PDBEntry;
import jalview.datamodel.HiddenColumns;
import jalview.datamodel.MappedFeatures;
import jalview.datamodel.PDBEntry;
@@
-871,21
+871,39
@@
public abstract class AAStructureBindingModel
{
AlignmentI alignment = view.getAlignment();
HiddenColumns hiddenCols = alignment.getHiddenColumns();
{
AlignmentI alignment = view.getAlignment();
HiddenColumns hiddenCols = alignment.getHiddenColumns();
-
/*
* 'matched' bit i will be set for visible alignment columns i where
* all sequences have a residue with a mapping to their PDB structure
*/
/*
* 'matched' bit i will be set for visible alignment columns i where
* all sequences have a residue with a mapping to their PDB structure
*/
- BitSet matched = new BitSet();
final int width = alignment.getWidth();
final int width = alignment.getWidth();
- for (int m = 0; m < width; m++)
+ BitSet matched = new BitSet();
+ ColumnSelection cs = view.getAlignViewport().getColumnSelection();
+ // restrict to active column selection, if there is one
+ if (cs != null && cs.hasSelectedColumns()
+ && cs.getSelected().size() >= 4)
{
{
- if (hiddenCols == null || hiddenCols.isVisible(m))
+ for (int s : cs.getSelected())
{
{
- matched.set(m);
+ if (hiddenCols == null)
+ {
+ matched.set(s);
+ }
+ else
+ {
+ matched.set(hiddenCols.visibleToAbsoluteColumn(s));
+ }
+ }
+ }
+ else
+ {
+ for (int m = 0; m < width; m++)
+ {
+ if (hiddenCols == null || hiddenCols.isVisible(m))
+ {
+ matched.set(m);
+ }
}
}
}
}
-
AAStructureBindingModel.SuperposeData[] structures = new AAStructureBindingModel.SuperposeData[files.length];
for (int f = 0; f < files.length; f++)
{
AAStructureBindingModel.SuperposeData[] structures = new AAStructureBindingModel.SuperposeData[files.length];
for (int f = 0; f < files.length; f++)
{
@@
-928,6
+946,8
@@
public abstract class AAStructureBindingModel
AtomSpecType backbone = structures[refStructure].isRna
? AtomSpecType.PHOSPHATE
: AtomSpecType.ALPHA;
AtomSpecType backbone = structures[refStructure].isRna
? AtomSpecType.PHOSPHATE
: AtomSpecType.ALPHA;
+ List<AtomSpecModel> models = new ArrayList<AtomSpecModel>();
+ models.add(refAtoms);
/*
* superpose each (other) structure to the reference in turn
*/
/*
* superpose each (other) structure to the reference in turn
*/
@@
-948,10
+968,13
@@
public abstract class AAStructureBindingModel
error += "; " + reply;
}
}
error += "; " + reply;
}
}
+ models.add(atomSpec);
}
}
}
}
+ List<StructureCommandI> finalView = commandGenerator
+ .centerViewOn(models);
+ executeCommands(finalView, false, "Centered on Superposition");
}
}
-
return error;
}
return error;
}
@@
-1280,6
+1303,12
@@
public abstract class AAStructureBindingModel
@Override
public void updateColours(Object source)
{
@Override
public void updateColours(Object source)
{
+ if (getViewer() == null)
+ {
+ // can happen if a viewer was not instantiated or cleaned up and is still
+ // registered - mostly during tests
+ return;
+ }
AlignmentViewPanel ap = (AlignmentViewPanel) source;
// ignore events from panels not used to colour this view
if (!getViewer().isUsedForColourBy(ap))
AlignmentViewPanel ap = (AlignmentViewPanel) source;
// ignore events from panels not used to colour this view
if (!getViewer().isUsedForColourBy(ap))