import java.net.BindException;
import java.util.ArrayList;
import java.util.Collections;
-import java.util.Iterator;
import java.util.LinkedHashMap;
import java.util.List;
import java.util.Map;
import ext.edu.ucsf.rbvi.strucviz2.StructureManager.ModelType;
import jalview.api.AlignmentViewPanel;
import jalview.bin.Cache;
-import jalview.datamodel.AlignmentI;
import jalview.datamodel.PDBEntry;
import jalview.datamodel.SearchResultMatchI;
import jalview.datamodel.SearchResultsI;
String lastHighlightCommand;
/**
+ * Returns a model of the structure positions described by the Chimera format atomspec
+ * @param atomSpec
+ * @return
+ */
+ protected AtomSpec parseAtomSpec(String atomSpec)
+ {
+ return AtomSpec.fromChimeraAtomspec(atomSpec);
+ }
+
+ /**
* Open a PDB structure file in Chimera and set up mappings from Jalview.
*
* We check if the PDB model id is already loaded in Chimera, if so don't reopen
DataSourceType protocol)
{
super(ssm, pdbentry, sequenceIs, protocol);
- chimeraManager = new ChimeraManager(new StructureManager(true));
- chimeraManager.setChimeraX(ViewerType.CHIMERAX.equals(getViewerType()));
- setStructureCommands(new ChimeraCommands());
+ boolean chimeraX = ViewerType.CHIMERAX.equals(getViewerType());
+ chimeraManager = chimeraX ? new ChimeraXManager(new StructureManager(true)) : new ChimeraManager(new StructureManager(true));
+ setStructureCommands(chimeraX ? new ChimeraXCommands() : new ChimeraCommands());
}
@Override
try
{
chimeraListener = new ChimeraListener(this);
- chimeraManager.startListening(chimeraListener.getUri());
+ startListening(chimeraListener.getUri());
} catch (BindException e)
{
System.err.println(
chimeraListener.shutdown();
chimeraListener = null;
}
+
+ /*
+ * the following call is added to avoid a stack trace error in Chimera
+ * after "stop really" is sent; Chimera > 1.14 will not need it; see also
+ * http://plato.cgl.ucsf.edu/trac/chimera/ticket/17597
+ */
+ if (closeChimera && (getViewerType() == ViewerType.CHIMERA))
+ {
+ chimeraManager.getChimeraProcess().destroy();
+ }
+
chimeraManager.clearOnChimeraExit();
chimeraManager = null;
}
if (getResponse)
{
reply = lastReply;
- Cache.log.debug(
- "Response from command ('" + cmd + "') was:\n" + lastReply);
+ if (Cache.isDebugEnabled()) {
+ Cache.debug(
+ "Response from command ('" + cmd + "') was:\n" + lastReply);
+ }
+ }
+ else
+ {
+ if (Cache.isDebugEnabled())
+ {
+ Cache.debug("Command executed: " + cmd);
+ }
}
return reply;
{
return;
}
-
+ if (!found)
+ {
+ // not a valid residue label command, so clear
+ cmd.setLength(0);
+ }
/*
- * unshow the label for the previous residue
+ * prepend with command
+ * to unshow the label for the previous residue
*/
if (lastHighlightCommand != null)
{
- chimeraManager.sendChimeraCommand("~" + lastHighlightCommand, false);
+ cmd.insert(0, ";");
+ cmd.insert(0,lastHighlightCommand);
+ cmd.insert(0,"~");
+
}
- if (found)
- {
- chimeraManager.sendChimeraCommand(command, false);
+ if (cmd.length()>0) {
+ executeCommand(true, null, new StructureCommand(cmd.toString()));
+ }
+
+ if (found) {
+ this.lastHighlightCommand = command;
}
- this.lastHighlightCommand = command;
}
/**
/*
* Ask Chimera for its current selection
*/
- List<String> selection = chimeraManager.getSelectedResidueSpecs();
+ StructureCommandI command = getCommandGenerator().getSelectedResidues();
+
+ Runnable action = new Runnable()
+ {
+ @Override
+ public void run()
+ {
+ List<String> chimeraReply = executeCommand(command, true);
+
+ List<String> selectedResidues = new ArrayList<>();
+ if (chimeraReply != null)
+ {
+ /*
+ * expect 0, 1 or more lines of the format either
+ * Chimera:
+ * residue id #0:43.A type GLY
+ * ChimeraX:
+ * residue id /A:89 name THR index 88
+ * We are only interested in the atomspec (third token of the reply)
+ */
+ for (String inputLine : chimeraReply)
+ {
+ String[] inputLineParts = inputLine.split("\\s+");
+ if (inputLineParts.length >= 5)
+ {
+ selectedResidues.add(inputLineParts[2]);
+ }
+ }
+ }
- /*
- * Parse model number, residue and chain for each selected position,
- * formatted as #0:123.A or #1.2:87.B (#model.submodel:residue.chain)
- */
- List<AtomSpec> atomSpecs = convertStructureResiduesToAlignment(
- selection);
+ /*
+ * Parse model number, residue and chain for each selected position,
+ * formatted as #0:123.A or #1.2:87.B (#model.submodel:residue.chain)
+ */
+ List<AtomSpec> atomSpecs = convertStructureResiduesToAlignment(
+ selectedResidues);
- /*
- * Broadcast the selection (which may be empty, if the user just cleared all
- * selections)
- */
- getSsm().mouseOverStructure(atomSpecs);
+ /*
+ * Broadcast the selection (which may be empty, if the user just cleared all
+ * selections)
+ */
+ getSsm().mouseOverStructure(atomSpecs);
+
+ }
+ };
+ new Thread(action).start();
}
/**
- * Converts a list of Chimera atomspecs to a list of AtomSpec representing the
+ * Converts a list of Chimera(X) atomspecs to a list of AtomSpec representing the
* corresponding residues (if any) in Jalview
*
* @param structureSelection
protected List<AtomSpec> convertStructureResiduesToAlignment(
List<String> structureSelection)
{
- boolean chimeraX = chimeraManager.isChimeraX();
List<AtomSpec> atomSpecs = new ArrayList<>();
for (String atomSpec : structureSelection)
{
try
{
- AtomSpec spec = AtomSpec.fromChimeraAtomspec(atomSpec, chimeraX);
+ AtomSpec spec = parseAtomSpec(atomSpec);
String pdbfilename = getPdbFileForModel(spec.getModelNumber());
spec.setPdbFile(pdbfilename);
atomSpecs.add(spec);
} catch (IllegalArgumentException e)
{
- Cache.log.error("Failed to parse atomspec: " + atomSpec);
+ Cache.error("Failed to parse atomspec: " + atomSpec);
}
}
return atomSpecs;
}
/**
- * Send a 'show' command for all atoms in the currently selected columns
- *
- * TODO: pull up to abstract structure viewer interface
- *
- * @param vp
- */
- public void highlightSelection(AlignmentViewPanel vp)
- {
- List<Integer> cols = vp.getAlignViewport().getColumnSelection()
- .getSelected();
- AlignmentI alignment = vp.getAlignment();
- StructureSelectionManager sm = getSsm();
- for (SequenceI seq : alignment.getSequences())
- {
- /*
- * convert selected columns into sequence positions
- */
- int[] positions = new int[cols.size()];
- int i = 0;
- for (Integer col : cols)
- {
- positions[i++] = seq.findPosition(col);
- }
- sm.highlightStructure(this, seq, positions);
- }
- }
-
- /**
* Constructs and send commands to Chimera to set attributes on residues for
* features visible in Jalview.
* <p>
}
else
{
- for (StructureCommandI command : commands)
- {
- sendAsynchronousCommand(command, null);
- }
+ executeCommands(commands, false, null);
}
return commands.size();
}
String path = tmp.getAbsolutePath();
StructureCommandI command = getCommandGenerator()
.openCommandFile(path);
- sendAsynchronousCommand(command, null);
+ executeCommand(false, null, command);
} catch (IOException e)
{
System.err.println("Sending commands to Chimera via file failed with "
}
/**
- * Get Chimera residues which have the named attribute, find the mapped
- * positions in the Jalview sequence(s), and set as sequence features
- *
- * @param attName
- * @param alignmentPanel
- */
- public void copyStructureAttributesToFeatures(String attName,
- AlignmentViewPanel alignmentPanel)
- {
- // todo pull up to AAStructureBindingModel (and interface?)
-
- /*
- * ask Chimera to list residues with the attribute, reporting its value
- */
- // this alternative command
- // list residues spec ':*/attName' attr attName
- // doesn't report 'None' values (which is good), but
- // fails for 'average.bfactor' (which is bad):
-
- String cmd = "list residues attr '" + attName + "'";
- List<String> residues = executeCommand(new StructureCommand(cmd), true);
-
- boolean featureAdded = createFeaturesForAttributes(attName, residues);
- if (featureAdded)
- {
- alignmentPanel.getFeatureRenderer().featuresAdded();
- }
- }
-
- /**
* Create features in Jalview for the given attribute name and structure
* residues.
*
*
* @param attName
* @param residues
- * @return
+ * @return the number of features added
*/
- protected boolean createFeaturesForAttributes(String attName,
+ protected int createFeaturesForAttributes(String attName,
List<String> residues)
{
- boolean featureAdded = false;
+ int featuresAdded = 0;
String featureGroup = getViewerFeatureGroup();
- boolean chimeraX = chimeraManager.isChimeraX();
for (String residue : residues)
{
try
{
- spec = AtomSpec.fromChimeraAtomspec(atomSpec, chimeraX);
+ spec = parseAtomSpec(atomSpec);
} catch (IllegalArgumentException e)
{
- System.err.println("Problem parsing atomspec " + atomSpec);
+ Cache.error("Problem parsing atomspec " + atomSpec);
continue;
}
start, end, score, featureGroup);
// todo: should SequenceFeature have an explicit property for chain?
// note: repeating the action shouldn't duplicate features
- featureAdded |= seq.addSequenceFeature(sf);
+ if (seq.addSequenceFeature(sf))
+ {
+ featuresAdded++;
+ }
}
}
- return featureAdded;
+ return featuresAdded;
}
/**
*/
public List<String> getChimeraAttributes()
{
- List<String> atts = chimeraManager.getAttrList();
- Iterator<String> it = atts.iterator();
- while (it.hasNext())
+ List<String> attributes = new ArrayList<>();
+ StructureCommandI command = getCommandGenerator().listResidueAttributes();
+ final List<String> reply = executeCommand(command, true);
+ if (reply != null)
{
- if (it.next().startsWith(ChimeraCommands.NAMESPACE_PREFIX))
+ for (String inputLine : reply)
{
- /*
- * attribute added from Jalview - exclude it
- */
- it.remove();
+ String[] lineParts = inputLine.split("\\s");
+ if (lineParts.length == 2 && lineParts[0].equals("resattr"))
+ {
+ String attName = lineParts[1];
+ /*
+ * exclude attributes added from Jalview
+ */
+ if (!attName.startsWith(ChimeraCommands.NAMESPACE_PREFIX))
+ {
+ attributes.add(attName);
+ }
+ }
}
}
- return atts;
+ return attributes;
}
/**
git://source.jalview.org / jalview.git / blobdiff