Fix off-by-one error between user orbital numbers and internal idx

Signed-off-by: Geoff Hutchison <[email protected]>
OpenChemistry · Oct 18, 2023 · f6b2ff6 · f6b2ff6
1 parent ccebc56
commit f6b2ff6
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/avogadro/qtplugins/surfaces/surfaces.cpp b/avogadro/qtplugins/surfaces/surfaces.cpp
@@ -143,8 +143,7 @@ bool Surfaces::handleCommand(const QString& command, const QVariantMap& options)
     if (ok)
       cubeResolution = res;
   }
-  if (options.contains("isovalue") &&
-      options["isovalue"].canConvert<float>()) {
+  if (options.contains("isovalue") && options["isovalue"].canConvert<float>()) {
     bool ok;
     float iso = options["isovalue"].toFloat(&ok);
     if (ok)
@@ -181,9 +180,11 @@ bool Surfaces::handleCommand(const QString& command, const QVariantMap& options)
         if (ok)
           index = index + n;
       }
-
-    } else
-      index = options.value("index").toInt();
+      index = index - 1; // start from zero
+    } else {
+      // internally, we count orbitals from zero
+      index = options.value("index").toInt() - 1;
+    }
   }
   bool beta = false;
   if (options.contains("spin")) {