JAL-2422 save/restore ChimeraX session
[jalview.git] / src / jalview / ext / rbvi / chimera / JalviewChimeraBinding.java
index 870c4fe..786fc6c 100644 (file)
@@ -31,6 +31,8 @@ import jalview.datamodel.SearchResultMatchI;
 import jalview.datamodel.SearchResultsI;
 import jalview.datamodel.SequenceFeature;
 import jalview.datamodel.SequenceI;
+import jalview.gui.Preferences;
+import jalview.gui.StructureViewer.ViewerType;
 import jalview.httpserver.AbstractRequestHandler;
 import jalview.io.DataSourceType;
 import jalview.schemes.ColourSchemeI;
@@ -51,6 +53,7 @@ import java.util.ArrayList;
 import java.util.BitSet;
 import java.util.Collections;
 import java.util.Hashtable;
+import java.util.Iterator;
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
@@ -76,10 +79,10 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
 
   private static final String ALPHACARBON = "CA";
 
-  private List<String> chainNames = new ArrayList<String>();
+  private List<String> chainNames = new ArrayList<>();
+
+  private Hashtable<String, String> chainFile = new Hashtable<>();
 
-  private Hashtable<String, String> chainFile = new Hashtable<String, String>();
-  
   /*
    * Object through which we talk to Chimera
    */
@@ -106,7 +109,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   /*
    * Map of ChimeraModel objects keyed by PDB full local file name
    */
-  private Map<String, List<ChimeraModel>> chimeraMaps = new LinkedHashMap<String, List<ChimeraModel>>();
+  private Map<String, List<ChimeraModel>> chimeraMaps = new LinkedHashMap<>();
 
   String lastHighlightCommand;
 
@@ -122,8 +125,8 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   /**
    * 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 it. This is the case if Chimera has opened a saved session file.
+   * We check if the PDB model id is already loaded in Chimera, if so don't reopen
+   * it. This is the case if Chimera has opened a saved session file.
    * 
    * @param pe
    * @return
@@ -133,8 +136,9 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
     String file = pe.getFile();
     try
     {
-      List<ChimeraModel> modelsToMap = new ArrayList<ChimeraModel>();
-      List<ChimeraModel> oldList = viewer.getModelList();
+      List<ChimeraModel> modelsToMap = new ArrayList<>();
+      List<ChimeraModel> oldList = viewer.isChimeraX() ? new ArrayList<>()
+              : viewer.getModelList();
       boolean alreadyOpen = false;
 
       /*
@@ -156,13 +160,32 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       if (!alreadyOpen)
       {
         viewer.openModel(file, pe.getId(), ModelType.PDB_MODEL);
-        List<ChimeraModel> newList = viewer.getModelList();
-        // JAL-1728 newList.removeAll(oldList) does not work
-        for (ChimeraModel cm : newList)
+        if (viewer.isChimeraX())
         {
-          if (cm.getModelName().equals(pe.getId()))
+          /*
+           * ChimeraX hack: force chimera model name to pdbId
+           */
+          int modelNumber = chimeraMaps.size() + 1;
+          String command = "setattr #" + modelNumber + " models name "
+                  + pe.getId();
+          sendChimeraCommand(command, false);
+          modelsToMap.add(new ChimeraModel(pe.getId(), ModelType.PDB_MODEL,
+                  modelNumber, 0));
+        }
+        else
+        {
+          /*
+           * Chimera: query for actual models and find the one with
+           * matching model name - set in viewer.openModel()
+           */
+          List<ChimeraModel> newList = viewer.getModelList();
+          // JAL-1728 newList.removeAll(oldList) does not work
+          for (ChimeraModel cm : newList)
           {
-            modelsToMap.add(cm);
+            if (cm.getModelName().equals(pe.getId()))
+            {
+              modelsToMap.add(cm);
+            }
           }
         }
       }
@@ -192,16 +215,20 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
    * @param protocol
    */
   public JalviewChimeraBinding(StructureSelectionManager ssm,
-          PDBEntry[] pdbentry, SequenceI[][] sequenceIs, DataSourceType protocol)
+          PDBEntry[] pdbentry, SequenceI[][] sequenceIs,
+          DataSourceType protocol)
   {
     super(ssm, pdbentry, sequenceIs, protocol);
     viewer = new ChimeraManager(new StructureManager(true));
+    String viewerType = Cache.getProperty(Preferences.STRUCTURE_DISPLAY);
+    viewer.setChimeraX(ViewerType.CHIMERAX.name().equals(viewerType));
+
   }
 
   /**
-   * Starts a thread that waits for the Chimera process to finish, so that we
-   * can then close the associated resources. This avoids leaving orphaned
-   * Chimera viewer panels in Jalview if the user closes Chimera.
+   * Starts a thread that waits for the Chimera process to finish, so that we can
+   * then close the associated resources. This avoids leaving orphaned Chimera
+   * viewer panels in Jalview if the user closes Chimera.
    */
   protected void startChimeraProcessMonitor()
   {
@@ -230,8 +257,8 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   }
 
   /**
-   * Start a dedicated HttpServer to listen for Chimera notifications, and tell
-   * it to start listening
+   * Start a dedicated HttpServer to listen for Chimera notifications, and tell it
+   * to start listening
    */
   public void startChimeraListener()
   {
@@ -241,8 +268,8 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       viewer.startListening(chimeraListener.getUri());
     } catch (BindException e)
     {
-      System.err.println("Failed to start Chimera listener: "
-              + e.getMessage());
+      System.err.println(
+              "Failed to start Chimera listener: " + e.getMessage());
     }
   }
 
@@ -263,8 +290,8 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
     for (String chain : toshow)
     {
       int modelNumber = getModelNoForChain(chain);
-      String showChainCmd = modelNumber == -1 ? "" : modelNumber + ":."
-              + chain.split(":")[1];
+      String showChainCmd = modelNumber == -1 ? ""
+              : modelNumber + ":." + chain.split(":")[1];
       if (!first)
       {
         cmd.append(",");
@@ -289,7 +316,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
    */
   public void closeViewer(boolean closeChimera)
   {
-    getSsm().removeStructureViewerListener(this, this.getPdbFile());
+    getSsm().removeStructureViewerListener(this, this.getStructureFiles());
     if (closeChimera)
     {
       viewer.exitChimera();
@@ -328,7 +355,9 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   public void colourByCharge()
   {
     colourBySequence = false;
-    String command = "color white;color red ::ASP;color red ::GLU;color blue ::LYS;color blue ::ARG;color yellow ::CYS";
+    String command = viewer.isChimeraX()
+            ? "color white;color :ASP,GLU red;color :LYS,ARG blue;color :CYS yellow"
+            : "color white;color red ::ASP;color red ::GLU;color blue ::LYS;color blue ::ARG;color yellow ::CYS";
     sendAsynchronousCommand(command, COLOURING_CHIMERA);
   }
 
@@ -340,7 +369,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
           int[] _refStructure, HiddenColumns[] _hiddenCols)
   {
     StringBuilder allComs = new StringBuilder(128);
-    String[] files = getPdbFile();
+    String[] files = getStructureFiles();
 
     if (!waitForFileLoad(files))
     {
@@ -349,6 +378,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
 
     refreshPdbEntries();
     StringBuilder selectioncom = new StringBuilder(256);
+    boolean chimeraX = viewer.isChimeraX();
     for (int a = 0; a < _alignment.length; a++)
     {
       int refStructure = _refStructure[a];
@@ -409,10 +439,16 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       String[] selcom = new String[files.length];
       for (int pdbfnum = 0; pdbfnum < files.length; pdbfnum++)
       {
+        final int modelNo = pdbfnum + (chimeraX ? 1 : 0);
+        // todo correct resolution to model number
         String chainCd = "." + structures[pdbfnum].chain;
         int lpos = -1;
         boolean run = false;
         StringBuilder molsel = new StringBuilder();
+        if (chimeraX)
+        {
+          molsel.append("/" + structures[pdbfnum].chain + ":");
+        }
 
         int nextColumnMatch = matched.nextSetBit(0);
         while (nextColumnMatch != -1)
@@ -426,7 +462,10 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
             if (lpos != -1)
             {
               molsel.append(String.valueOf(lpos));
-              molsel.append(chainCd);
+              if (!chimeraX)
+              {
+                molsel.append(chainCd);
+              }
               molsel.append(",");
             }
             run = false;
@@ -456,18 +495,24 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
         if (lpos != -1)
         {
           molsel.append(String.valueOf(lpos));
-          molsel.append(chainCd);
+          if (!chimeraX)
+          {
+            molsel.append(chainCd);
+          }
         }
         if (molsel.length() > 1)
         {
           selcom[pdbfnum] = molsel.toString();
-          selectioncom.append("#").append(String.valueOf(pdbfnum))
-                  .append(":");
+          selectioncom.append("#").append(String.valueOf(modelNo));
+          if (!chimeraX)
+          {
+            selectioncom.append(":");
+          }
           selectioncom.append(selcom[pdbfnum]);
-          selectioncom.append(" ");
+          // selectioncom.append(" ");
           if (pdbfnum < files.length - 1)
           {
-            selectioncom.append("| ");
+            selectioncom.append("|");
           }
         }
         else
@@ -479,6 +524,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       StringBuilder command = new StringBuilder(256);
       for (int pdbfnum = 0; pdbfnum < files.length; pdbfnum++)
       {
+        final int modelNo = pdbfnum + (chimeraX ? 1 : 0);
         if (pdbfnum == refStructure || selcom[pdbfnum] == null
                 || selcom[refStructure] == null)
         {
@@ -498,29 +544,57 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
          * @see
          * https://www.cgl.ucsf.edu/chimera/docs/UsersGuide/midas/match.html
          */
-        command.append("match ").append(getModelSpec(pdbfnum)).append(":");
+        command.append(chimeraX ? "align " : "match ");
+        command.append(getModelSpec(modelNo));
+        if (!chimeraX)
+        {
+          command.append(":");
+        }
         command.append(selcom[pdbfnum]);
         command.append("@").append(
                 structures[pdbfnum].isRna ? PHOSPHORUS : ALPHACARBON);
-        // JAL-1757 exclude alternate CA locations
-        command.append(NO_ALTLOCS);
-        command.append(" ").append(getModelSpec(refStructure)).append(":");
+        // JAL-1757 exclude alternate CA locations - ChimeraX syntax tbd
+        if (!chimeraX)
+        {
+          command.append(NO_ALTLOCS);
+        }
+        command.append(chimeraX ? " toAtoms " : " ")
+                .append(getModelSpec(refStructure + (chimeraX ? 1 : 0)));
+        if (!chimeraX)
+        {
+          command.append(":");
+        }
         command.append(selcom[refStructure]);
         command.append("@").append(
                 structures[refStructure].isRna ? PHOSPHORUS : ALPHACARBON);
-        command.append(NO_ALTLOCS);
+        if (!chimeraX)
+        {
+          command.append(NO_ALTLOCS);
+        }
       }
       if (selectioncom.length() > 0)
       {
         if (debug)
         {
           System.out.println("Select regions:\n" + selectioncom.toString());
-          System.out.println("Superimpose command(s):\n"
-                  + command.toString());
+          System.out.println(
+                  "Superimpose command(s):\n" + command.toString());
+        }
+        // allComs.append("~display all; ");
+        // if (chimeraX)
+        // {
+        // allComs.append("show ").append(selectioncom.toString())
+        // .append(" pbonds");
+        // }
+        // else
+        // {
+        // allComs.append("chain @CA|P; ribbon ");
+        // allComs.append(selectioncom.toString());
+        // }
+        if (allComs.length() > 0) {
+          allComs.append(";");
         }
-        allComs.append("~display all; chain @CA|P; ribbon ")
-                .append(selectioncom.toString())
-                .append(";" + command.toString());
+        allComs.append(command.toString());
       }
     }
 
@@ -536,8 +610,19 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       {
         System.out.println("Select regions:\n" + selectioncom.toString());
       }
-      allComs.append("; ~display all; chain @CA|P; ribbon ")
-              .append(selectioncom.toString()).append("; focus");
+      allComs.append(";~display all; ");
+      if (chimeraX)
+      {
+        allComs.append("show @CA|P pbonds; show ")
+                .append(selectioncom.toString()).append(" ribbons; view");
+      }
+      else
+      {
+        allComs.append("chain @CA|P; ribbon ; focus");
+        allComs.append(selectioncom.toString());
+      }
+      // allComs.append("; ~display all; chain @CA|P; ribbon ")
+      // .append(selectioncom.toString()).append("; focus");
       List<String> chimeraReplies = sendChimeraCommand(allComs.toString(),
               true);
       for (String reply : chimeraReplies)
@@ -568,7 +653,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   {
     if (pdbfnum < 0 || pdbfnum >= getPdbCount())
     {
-      return "";
+      return "#" + pdbfnum; // temp hack for ChimeraX
     }
 
     /*
@@ -577,7 +662,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
      * to the Chimera command 'list models type molecule', see
      * ChimeraManager.getModelList().
      */
-    List<ChimeraModel> maps = chimeraMaps.get(getPdbFile()[pdbfnum]);
+    List<ChimeraModel> maps = chimeraMaps.get(getStructureFiles()[pdbfnum]);
     boolean hasSubModels = maps != null && maps.size() > 1;
     return "#" + String.valueOf(pdbfnum) + (hasSubModels ? ".1" : "");
   }
@@ -596,8 +681,8 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       return true;
     }
 
-    boolean launched = viewer.launchChimera(StructureManager
-            .getChimeraPaths());
+    boolean launched = viewer.launchChimera(
+            StructureManager.getChimeraPaths(viewer.isChimeraX()));
     if (launched)
     {
       startChimeraProcessMonitor();
@@ -698,7 +783,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
           String[] files, SequenceRenderer sr, AlignmentViewPanel viewPanel)
   {
     return ChimeraCommands.getColourBySequenceCommand(getSsm(), files,
-            getSequence(), sr, viewPanel);
+            getSequence(), sr, viewPanel, viewer.isChimeraX());
   }
 
   /**
@@ -741,25 +826,23 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
    */
   private int _modelFileNameMap[];
 
-
   // ////////////////////////////////
   // /StructureListener
   @Override
-  public synchronized String[] getPdbFile()
+  public synchronized String[] getStructureFiles()
   {
     if (viewer == null)
     {
       return new String[0];
     }
 
-    return chimeraMaps.keySet().toArray(
-            modelFileNames = new String[chimeraMaps.size()]);
+    return chimeraMaps.keySet()
+            .toArray(modelFileNames = new String[chimeraMaps.size()]);
   }
 
   /**
-   * Construct and send a command to highlight zero, one or more atoms. We do
-   * this by sending an "rlabel" command to show the residue label at that
-   * position.
+   * Construct and send a command to highlight zero, one or more atoms. We do this
+   * by sending an "rlabel" command to show the residue label at that position.
    */
   @Override
   public void highlightAtoms(List<AtomSpec> atoms)
@@ -769,6 +852,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       return;
     }
 
+    boolean forChimeraX = viewer.isChimeraX();
     StringBuilder cmd = new StringBuilder(128);
     boolean first = true;
     boolean found = false;
@@ -783,18 +867,26 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       {
         if (first)
         {
-          cmd.append("rlabel #").append(cms.get(0).getModelNumber())
-                  .append(":");
+          cmd.append(forChimeraX ? "label #" : "rlabel #");
         }
         else
         {
           cmd.append(",");
         }
         first = false;
-        cmd.append(pdbResNum);
-        if (!chain.equals(" "))
+        if (forChimeraX)
+        {
+          cmd.append(cms.get(0).getModelNumber())
+                  .append("/").append(chain).append(":").append(pdbResNum);
+        }
+        else
         {
-          cmd.append(".").append(chain);
+          cmd.append(cms.get(0).getModelNumber())
+                  .append(":").append(pdbResNum);
+          if (!chain.equals(" ") && !forChimeraX)
+          {
+            cmd.append(".").append(chain);
+          }
         }
         found = true;
       }
@@ -837,7 +929,8 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
      * 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);
+    List<AtomSpec> atomSpecs = convertStructureResiduesToAlignment(
+            selection);
 
     /*
      * Broadcast the selection (which may be empty, if the user just cleared all
@@ -856,12 +949,13 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   protected List<AtomSpec> convertStructureResiduesToAlignment(
           List<String> structureSelection)
   {
-    List<AtomSpec> atomSpecs = new ArrayList<AtomSpec>();
+    boolean chimeraX = viewer.isChimeraX();
+    List<AtomSpec> atomSpecs = new ArrayList<>();
     for (String atomSpec : structureSelection)
     {
       try
       {
-        AtomSpec spec = AtomSpec.fromChimeraAtomspec(atomSpec);
+        AtomSpec spec = AtomSpec.fromChimeraAtomspec(atomSpec, chimeraX);
         String pdbfilename = getPdbFileForModel(spec.getModelNumber());
         spec.setPdbFile(pdbfilename);
         atomSpecs.add(spec);
@@ -923,21 +1017,35 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       return;
     }
 
-    // Chimera expects RBG values in the range 0-1
-    final double normalise = 255D;
     viewerCommandHistory(false);
     StringBuilder command = new StringBuilder(128);
 
     List<String> residueSet = ResidueProperties.getResidues(isNucleotide(),
             false);
+
+    /*
+     * concatenate colour commands, one per residue symbol
+     * Chimera format:  color 0.000000,0.372549,0.627451 ::VAL
+     * ChimeraX format: color :VAL rgb(73,73,182)
+     */
+    boolean chimeraX = viewer.isChimeraX();
     for (String resName : residueSet)
     {
-      char res = resName.length() == 3 ? ResidueProperties
-              .getSingleCharacterCode(resName) : resName.charAt(0);
+      char res = resName.length() == 3
+              ? ResidueProperties.getSingleCharacterCode(resName)
+              : resName.charAt(0);
       Color col = cs.findColour(res, 0, null, null, 0f);
-      command.append("color " + col.getRed() / normalise + ","
-              + col.getGreen() / normalise + "," + col.getBlue()
-              / normalise + " ::" + resName + ";");
+      command.append("color ");
+      String colorSpec = getRgbDescriptor(col, chimeraX);
+      if (chimeraX)
+      {
+        command.append(":").append(resName).append(" ").append(colorSpec);
+      }
+      else
+      {
+        command.append(colorSpec).append(" ::").append(resName);
+      }
+      command.append(";");
     }
 
     sendAsynchronousCommand(command.toString(), COLOURING_CHIMERA);
@@ -983,7 +1091,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   /**
    * Send the Chimera 'background solid <color>" command.
    * 
-   * @see https 
+   * @see https
    *      ://www.cgl.ucsf.edu/chimera/current/docs/UsersGuide/midas/background
    *      .html
    * @param col
@@ -992,15 +1100,40 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   public void setBackgroundColour(Color col)
   {
     viewerCommandHistory(false);
-    double normalise = 255D;
-    final String command = "background solid " + col.getRed() / normalise
-            + "," + col.getGreen() / normalise + "," + col.getBlue()
-            / normalise + ";";
+    String command = "set bgColor "
+            + getRgbDescriptor(col, viewer.isChimeraX());
     viewer.sendChimeraCommand(command, false);
     viewerCommandHistory(true);
   }
 
   /**
+   * Answers the Chimera/X format for RGB values of the given colour.
+   * 
+   * <pre>
+   * Chimera: r,g,b with values scaled [0=1]
+   * ChimeraX: rgb(r,g,b) with values scaled 0-255
+   * </pre>
+   * 
+   * @param col
+   * @param chimeraX
+   * @return
+   */
+  private static String getRgbDescriptor(Color col, boolean chimeraX)
+  {
+    if (chimeraX)
+    {
+      return String.format("rgb(%d,%d,%d)", col.getRed(), col.getGreen(),
+              col.getBlue());
+    }
+    else
+    {
+      double scale = 255D;
+      return String.format("%f,%f,%f", col.getRed() / scale,
+              col.getGreen() / scale, col.getBlue() / scale);
+    }
+  }
+
+  /**
    * Ask Chimera to save its session to the given file. Returns true if
    * successful, else false.
    * 
@@ -1011,7 +1144,12 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   {
     if (isChimeraRunning())
     {
-      List<String> reply = viewer.sendChimeraCommand("save " + filepath,
+      /*
+       * Chimera:  https://www.cgl.ucsf.edu/chimera/current/docs/UsersGuide/midas/save.html
+       * ChimeraX: https://www.cgl.ucsf.edu/chimerax/docs/user/commands/save.html
+       */
+      String command = isChimeraX() ? "save session " : "save ";
+      List<String> reply = viewer.sendChimeraCommand(command + filepath,
               true);
       if (reply.contains("Session written"))
       {
@@ -1028,13 +1166,18 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
 
   /**
    * Ask Chimera to open a session file. Returns true if successful, else false.
-   * The filename must have a .py extension for this command to work.
+   * The filename must have a .py (Chimera) or .cxs (ChimeraX) extension for
+   * this command to work.
    * 
    * @param filepath
    * @return
    */
   public boolean openSession(String filepath)
   {
+    /*
+     * Chimera:  https://www.cgl.ucsf.edu/chimera/current/docs/UsersGuide/midas/open.html
+     * ChimeraX: https://www.cgl.ucsf.edu/chimerax/docs/user/commands/open.html
+     */
     sendChimeraCommand("open " + filepath, true);
     // todo: test for failure - how?
     return true;
@@ -1057,7 +1200,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
    */
   public void focusView()
   {
-    sendChimeraCommand("focus", false);
+    sendChimeraCommand(viewer.isChimeraX() ? "view" : "focus", false);
   }
 
   /**
@@ -1100,7 +1243,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
     // TODO refactor as required to pull up to an interface
     AlignmentI alignment = avp.getAlignment();
 
-    String[] files = getPdbFile();
+    String[] files = getStructureFiles();
     if (files == null)
     {
       return 0;
@@ -1108,7 +1251,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
 
     StructureMappingcommandSet commandSet = ChimeraCommands
             .getSetAttributeCommandsForFeatures(getSsm(), files,
-                    getSequence(), avp);
+                    getSequence(), avp, viewer.isChimeraX());
     String[] commands = commandSet.commands;
     if (commands.length > 10)
     {
@@ -1125,17 +1268,18 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   }
 
   /**
-   * Write commands to a temporary file, and send a command to Chimera to open
-   * the file as a commands script. For use when sending a large number of
-   * separate commands would overload the REST interface mechanism.
+   * Write commands to a temporary file, and send a command to Chimera to open the
+   * file as a commands script. For use when sending a large number of separate
+   * commands would overload the REST interface mechanism.
    * 
    * @param commands
    */
   protected void sendCommandsByFile(String[] commands)
   {
+    boolean toChimeraX = viewer.isChimeraX();
     try
     {
-      File tmp = File.createTempFile("chim", ".com");
+      File tmp = File.createTempFile("chim", toChimeraX ? ".cxc" : ".com");
       tmp.deleteOnExit();
       PrintWriter out = new PrintWriter(new FileOutputStream(tmp));
       for (String command : commands)
@@ -1145,12 +1289,12 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
       out.flush();
       out.close();
       String path = tmp.getAbsolutePath();
-      sendAsynchronousCommand("open cmd:" + path, null);
+      String command = "open " + (toChimeraX ? "" : "cmd:") + path;
+      sendAsynchronousCommand(command, null);
     } catch (IOException e)
     {
-      System.err
-              .println("Sending commands to Chimera via file failed with "
-                      + e.getMessage());
+      System.err.println("Sending commands to Chimera via file failed with "
+              + e.getMessage());
     }
   }
 
@@ -1204,6 +1348,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
   {
     boolean featureAdded = false;
     String featureGroup = getViewerFeatureGroup();
+    boolean chimeraX = viewer.isChimeraX();
 
     for (String residue : residues)
     {
@@ -1227,7 +1372,7 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
 
       try
       {
-        spec = AtomSpec.fromChimeraAtomspec(atomSpec);
+        spec = AtomSpec.fromChimeraAtomspec(atomSpec, chimeraX);
       } catch (IllegalArgumentException e)
       {
         System.err.println("Problem parsing atomspec " + atomSpec);
@@ -1288,7 +1433,6 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
     return CHIMERA_FEATURE_GROUP;
   }
 
-
   public Hashtable<String, String> getChainFile()
   {
     return chainFile;
@@ -1308,4 +1452,32 @@ public abstract class JalviewChimeraBinding extends AAStructureBindingModel
     }
     return -1;
   }
+
+  /**
+   * Answers a (possibly empty) list of attribute names in Chimera[X], excluding
+   * any which were added from Jalview
+   * 
+   * @return
+   */
+  public List<String> getChimeraAttributes()
+  {
+    List<String> atts = viewer.getAttrList();
+    Iterator<String> it = atts.iterator();
+    while (it.hasNext())
+    {
+      if (it.next().startsWith(ChimeraCommands.NAMESPACE_PREFIX))
+      {
+        /*
+         * attribute added from Jalview - exclude it
+         */
+        it.remove();
+      }
+    }
+    return atts;
+  }
+
+  public boolean isChimeraX()
+  {
+    return viewer.isChimeraX();
+  }
 }