Merge pull request #220 from prabhuramachandran/fix-viewer-indexing-e…

…rror Fix silly count offset-by-one error. [ci skip]
pypr · Jun 5, 2019 · 4c87d19 · 4c87d19
2 parents c77cd10 + e05fec5
commit 4c87d19
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pysph/tools/mayavi_viewer.py b/pysph/tools/mayavi_viewer.py
@@ -1055,7 +1055,7 @@ def _file_count_changed(self, value):
         if not self.files:
             return
         # Load the new file.
-        value = min(value, len(self.files))
+        value = min(value, len(self.files) - 1)
         fname = self.files[value]
         if not os.path.exists(fname):
             print("File %s is missing, ignoring!" % fname)
@@ -1143,7 +1143,7 @@ def _update_files_fired(self):
         sort_file_list(files)
         self.files = files
         if len(files) > 0:
-            fc = min(len(files), fc)
+            fc = min(len(files) - 1, fc)
             self.file_count = fc
             if self.play:
                 self._play_changed(self.play)
@@ -1168,7 +1168,7 @@ def _directory_changed(self, d):
             self._clear()
             sort_file_list(files)
             self.files = files
-            self.file_count = min(self.file_count, len(files))
+            self.file_count = min(self.file_count, len(files) - 1)
         else:
             pass
         config_file = os.path.join(d, 'mayavi_config.py')