boris-behav-obs 9.3.1__tar.gz → 9.3.3__tar.gz

{boris_behav_obs-9.3.1/boris_behav_obs.egg-info → boris_behav_obs-9.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: boris-behav-obs
-Version: 9.3.1
+Version: 9.3.3
 Summary: BORIS - Behavioral Observation Research Interactive Software
 Author-email: Olivier Friard <olivier.friard@unito.it>
 License-Expression: GPL-3.0-only

boris_behav_obs-9.3.3/boris/analysis_plugins/_latency.py

@@ -0,0 +1,59 @@
+"""
+BORIS plugin
+
+number of occurences of behaviors
+"""
+
+import pandas as pd
+
+__version__ = "0.0.1"
+__version_date__ = "2025-04-10"
+__plugin_name__ = "Behavior latency"
+__author__ = "Olivier Friard - University of Torino - Italy"
+
+
+import itertools
+
+
+def run(df: pd.DataFrame):
+    """
+    Latency of a behavior after another.
+    """
+
+    df["start_time"] = pd.to_datetime(df["Start (s)"])
+    df["end_time"] = pd.to_datetime(df["Stop (s)"])
+
+    latency_by_subject: dict = {}
+
+    for subject, group in df.groupby("subject"):
+        behaviors = group["behavior"].tolist()
+        # combinations = []
+        # Utiliser itertools pour créer des combinaisons 2 à 2 des comportements
+        for comb in itertools.combinations(behaviors, 2):
+            # combinations.append(comb)
+
+            last_A_end_time = None
+
+            # Liste pour stocker les latences de chaque sujet
+            subject_latency = []
+
+            for index, row in group.iterrows():
+                if row["behavior"] == comb[0]:
+                    # Si on rencontre un comportement A, on réinitialise le temps de fin du comportement A
+                    last_A_end_time = row["end_time"]
+                    subject_latency.append(None)  # Pas de latence pour A
+                elif row["behavior"] == comb[1] and last_A_end_time is not None:
+                    # Si on rencontre un comportement B et qu'on a déjà vu un A avant
+                    latency_time = row["start_time"] - last_A_end_time
+                    subject_latency.append(latency_time)
+                else:
+                    # Si on rencontre un B mais sans A avant
+                    subject_latency.append(None)
+
+            # Ajout des latences calculées au DataFrame
+            df.loc[group.index, f"latency {comb[1]} after {comb[0]}"] = subject_latency
+
+            # Calcul de la latence totale ou moyenne par sujet
+            latency_by_subject[(subject, comb)] = df.groupby("subject")["latency"].agg(["sum", "mean"])
+
+    return str(latency_by_subject)

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/behav_coding_map_creator.py

@@ -24,6 +24,7 @@ import binascii
 import io
 import json
 from pathlib import Path
+import gui_utilities
 
 from PySide6.QtCore import QBuffer, QByteArray, QIODevice, QLineF, QPoint, Qt, Signal
 from PySide6.QtGui import QBrush, QColor, QIcon, QMouseEvent, QPen, QPixmap, QPolygonF, QAction
@@ -790,7 +791,6 @@ class BehaviorsMapCreatorWindow(QMainWindow):
 
         if not self.fileName:
             return
-        """if os.path.splitext(self.fileName)[1] != ".behav_coding_map":"""
         if Path(self.fileName).suffix != ".behav_coding_map":
             self.fileName += ".behav_coding_map"
         self.saveMap()
@@ -1106,10 +1106,6 @@ if __name__ == "__main__":
     app = QApplication(sys.argv)
     window = BehaviorsMapCreatorWindow(["North zone", "East zone", "South zone", "West zone"])
     window.bcm_list = []
-    window.resize(cfg.CODING_MAP_RESIZE_W, cfg.CODING_MAP_RESIZE_H)
-    screen_geometry = app.primaryScreen().geometry()
-    center_x = (screen_geometry.width() - window.width()) // 2
-    center_y = (screen_geometry.height() - window.height()) // 2
-    window.move(center_x, center_y)
+    gui_utilities.resize_center(app, window, cfg.CODING_MAP_RESIZE_W, cfg.CODING_MAP_RESIZE_H)
     window.show()
     sys.exit(app.exec())

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/config.py

@@ -155,6 +155,7 @@ CHECK_PROJECT_INTEGRITY = "check_project_integrity"
 YES = "Yes"
 NO = "No"
 CANCEL = "Cancel"
+IGNORE = "Ignore"
 APPEND = "Append"
 CLOSE = "Close"
 REPLACE = "Replace"
@@ -454,7 +455,8 @@ POINT = "POINT"
 START = "START"
 STOP = "STOP"
 
-PLAYER1, PLAYER2 = "1", "2"
+PLAYER1 = "1"
+PLAYER2 = "2"
 ALL_PLAYERS = [str(x + 1) for x in range(N_PLAYER)]
 
 VISUALIZE_SPECTROGRAM = "visualize_spectrogram"
@@ -701,6 +703,7 @@ EMPTY_PROJECT = {
     ETHOGRAM: {},
     OBSERVATIONS: {},
     BEHAVIORAL_CATEGORIES: [],
+    BEHAVIORAL_CATEGORIES_CONF: {},
     INDEPENDENT_VARIABLES: {},
     CODING_MAP: {},
     BEHAVIORS_CODING_MAP: [],

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/core.py

@@ -114,7 +114,6 @@ from . import cmd_arguments
 
 from . import core_qrc
 from .core_ui import Ui_MainWindow
-import exifread
 from . import config as cfg
 from . import video_operations
 
@@ -135,8 +134,9 @@ __version__ = version.__version__
 __version_date__ = version.__version_date__
 
 # check minimal version of python
-if util.versiontuple(platform.python_version()) < util.versiontuple("3.8"):
-    msg = f"BORIS requires Python 3.8+! You are using Python v. {platform.python_version()}\n"
+MIN_PYTHON_VERSION = "3.12"
+if util.versiontuple(platform.python_version()) < util.versiontuple(MIN_PYTHON_VERSION):
+    msg = f"BORIS requires Python {MIN_PYTHON_VERSION}+! You are using Python v. {platform.python_version()}\n"
     logging.critical(msg)
     sys.exit()
 
@@ -237,19 +237,6 @@ class TableModel(QAbstractTableModel):
                         return self._data[row][event_idx]
 
 
-"""
-class ButtonEventFilter(QObject):
-    def eventFilter(self, obj, event):
-        print("event filter")
-        if isinstance(obj, QPushButton) and event.type() == QEvent.KeyPress:
-            print("keypress")
-            if event.key() in (Qt.Key_Enter, Qt.Key_Return, Qt.Key_Space):
-                print("enter sapce")
-                return False  # Block the event
-        return super().eventFilter(obj, event)
-"""
-
-
 class MainWindow(QMainWindow, Ui_MainWindow):
     """
     Main BORIS window
@@ -301,7 +288,7 @@ class MainWindow(QMainWindow, Ui_MainWindow):
     ext_data_timer_list: list = []
     projectFileName: str = ""
     mediaTotalLength = None
-    beep_every = 0
+    beep_every: float = 0.0
 
     plot_colors = cfg.BEHAVIORS_PLOT_COLORS
     behav_category_colors = cfg.CATEGORY_COLORS_LIST
@@ -315,19 +302,20 @@ class MainWindow(QMainWindow, Ui_MainWindow):
     fast = 10
 
     currentStates: dict = {}
-    subject_name_index = {}
+    subject_name_index: dict = {}
     flag_slow = False
     play_rate: float = 1
     play_rate_step: float = 0.1
     currentSubject: str = ""  # contains the current subject of observation
-    coding_map_window_geometry = 0
 
     # FFmpeg
-    memx, memy, mem_player = -1, -1, -1
+    memx = -1
+    memy = -1
+    mem_player = -1
 
     # path for ffmpeg/ffmpeg.exe program
-    ffmpeg_bin = ""
-    ffmpeg_cache_dir = ""
+    ffmpeg_bin: str = ""
+    ffmpeg_cache_dir: str = ""
 
     # dictionary for FPS storing
     fps = 0
@@ -390,9 +378,6 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         super(MainWindow, self).__init__(parent)
         self.setupUi(self)
 
-        # disable trigger with RETURN or SPACE keys
-        """filter_obj = ButtonEventFilter()
-        self.pb_live_obs.installEventFilter(filter_obj)"""
         self.pb_live_obs.setFocusPolicy(Qt.NoFocus)
 
         self.ffmpeg_bin = ffmpeg_bin
@@ -418,25 +403,18 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         self.tb_export.setMenu(self.menu)
         """
 
-        gui_utilities.set_icons(self, theme_mode=self.theme_mode())
+        gui_utilities.set_icons(self, theme_mode=gui_utilities.theme_mode())
 
         self.setWindowTitle(f"{cfg.programName} ({__version__})")
 
-        self.w_obs_info.setVisible(False)
-
         self.lbLogoBoris.setPixmap(QPixmap(":/logo"))
-
         self.lbLogoBoris.setScaledContents(False)
         self.lbLogoBoris.setAlignment(Qt.AlignCenter)
 
-        # self.lbLogoUnito.setPixmap(QPixmap(":/dbios_unito"))
-        # self.lbLogoUnito.setScaledContents(False)
-        # self.lbLogoUnito.setAlignment(Qt.AlignCenter)
-
         self.toolBar.setEnabled(True)
 
         # start with dock widget invisible
-        for w in [self.dwEvents, self.dwEthogram, self.dwSubjects]:
+        for w in (self.w_obs_info, self.dwEvents, self.dwEthogram, self.dwSubjects):
             w.setVisible(False)
             w.keyPressEvent = self.keyPressEvent
 
@@ -475,15 +453,6 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         self.lbTimeOffset.setMinimumWidth(160)
         self.statusbar.addPermanentWidget(self.lbTimeOffset)
 
-        # play rate are now displayed in the main info widget
-        """
-        # SPEED
-        self.lbSpeed = QLabel()
-        self.lbSpeed.setFrameStyle(QFrame.StyledPanel)
-        self.lbSpeed.setMinimumWidth(40)
-        self.statusbar.addPermanentWidget(self.lbSpeed)
-        """
-
         # set painter for tv_events to highlight current row
         delegate = self.CustomItemDelegate()
         self.tv_events.setItemDelegate(delegate)
@@ -499,14 +468,6 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         plugins.load_plugins(self)
         plugins.add_plugins_to_menu(self)
 
-    def theme_mode(self):
-        """
-        return the theme mode (dark or light) of the OS
-        """
-        palette = QApplication.instance().palette()
-        color = palette.window().color()
-        return "dark" if color.value() < 128 else "light"  # Dark mode if the color value is less than 128
-
     class CustomItemDelegate(QStyledItemDelegate):
         def paint(self, painter, option, index):
             # Custom drawing logic here (overriding paint)
@@ -1506,7 +1467,9 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         # one media
         if self.dw_player[player].player.playlist_count == 1:
             if new_time < self.dw_player[player].player.duration:
-                self.dw_player[player].player.seek(new_time, "absolute+exact")
+                new_time_float = round(float(new_time), 3)
+
+                self.dw_player[player].player.seek(new_time_float, "absolute+exact")
 
                 if player == 0 and not self.user_move_slider:
                     self.video_slider.setValue(
@@ -3835,7 +3798,7 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         if self.geometric_measurements_mode:
             geometric_measurement.redraw_measurements(self)
 
-            self.actionPlay.setIcon(QIcon(f":/play_{self.theme_mode()}"))
+            self.actionPlay.setIcon(QIcon(f":/play_{gui_utilities.theme_mode()}"))
 
     def previous_frame(self) -> None:
         """
@@ -3859,7 +3822,7 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         if self.geometric_measurements_mode:
             geometric_measurement.redraw_measurements(self)
 
-            self.actionPlay.setIcon(QIcon(f":/play_{self.theme_mode()}"))
+            self.actionPlay.setIcon(QIcon(f":/play_{gui_utilities.theme_mode()}"))
 
     def run_event_outside(self):
         """
@@ -4463,7 +4426,7 @@ class MainWindow(QMainWindow, Ui_MainWindow):
             for data_timer in self.ext_data_timer_list:
                 data_timer.stop()
 
-            self.actionPlay.setIcon(QIcon(f":/play_{self.theme_mode()}"))
+            self.actionPlay.setIcon(QIcon(f":/play_{gui_utilities.theme_mode()}"))
 
         if msg:
             self.lb_current_media_time.setText(msg)
@@ -5634,7 +5597,7 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         for data_timer in self.ext_data_timer_list:
             data_timer.start()
 
-        self.actionPlay.setIcon(QIcon(f":/pause_{self.theme_mode()}"))
+        self.actionPlay.setIcon(QIcon(f":/pause_{gui_utilities.theme_mode()}"))
         self.actionPlay.setText("Pause")
 
         return True
@@ -5668,7 +5631,7 @@ class MainWindow(QMainWindow, Ui_MainWindow):
         for idx in self.plot_data:
             self.timer_plot_data_out(self.plot_data[idx])
 
-        self.actionPlay.setIcon(QIcon(f":/play_{self.theme_mode()}"))
+        self.actionPlay.setIcon(QIcon(f":/play_{gui_utilities.theme_mode()}"))
         self.actionPlay.setText("Play")
 
     def play_activated(self):
@@ -5823,8 +5786,6 @@ class MainWindow(QMainWindow, Ui_MainWindow):
 
 
 def main():
-    # QApplication.setAttribute(Qt.AA_EnableHighDpiScaling)
-
     app = QApplication(sys.argv)
     app.setStyle("Fusion")
 
@@ -5917,7 +5878,7 @@ def main():
                 results.show()
 
     window.show()
-    window.raise_()
+    window.raise_()  # for overlapping widget (?)
 
     if observation_to_open and "error" not in pj:
         r = observation_operations.load_observation(window, obs_id=observation_to_open, mode=cfg.OBS_START)

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/dialog.py

@@ -71,7 +71,12 @@ from . import utilities as util
 
 def MessageDialog(title: str, text: str, buttons: tuple) -> str:
     """
-    generic message dialog
+    show a generic message dialog and returns the text of the clicked button
+
+    Args:
+        title (str): Title of the dialog box
+        text (str): text of the dialog box
+        buttons (tuple): text for buttons
 
     Return
         str: text of the clicked button
@@ -83,8 +88,8 @@ def MessageDialog(title: str, text: str, buttons: tuple) -> str:
     for button in buttons:
         message.addButton(button, QMessageBox.YesRole)
 
-    # message.setWindowFlags(Qt.WindowStaysOnTopHint)
-    message.exec_()
+    message.setWindowFlags(message.windowFlags() | Qt.WindowStaysOnTopHint)
+    message.exec()
     return message.clickedButton().text()
 
 

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/gui_utilities.py

@@ -22,10 +22,19 @@ Copyright 2012-2025 Olivier Friard
 import pathlib as pl
 import logging
 from PySide6.QtCore import QSettings
-from PySide6.QtWidgets import QWidget
+from PySide6.QtWidgets import QWidget, QApplication
 from PySide6.QtGui import QIcon
 
 
+def theme_mode() -> str:
+    """
+    return the theme mode (dark or light) of the OS
+    """
+    palette = QApplication.instance().palette()
+    color = palette.window().color()
+    return "dark" if color.value() < 128 else "light"  # Dark mode if the color value is less than 128
+
+
 def save_geometry(widget: QWidget, widget_name: str):
     """
     save window geometry in ini file
@@ -44,6 +53,7 @@ def restore_geometry(widget: QWidget, widget_name: str, default_width_height):
     """
     restore window geometry in ini file
     """
+
     def default_resize(widget, default_width_height):
         if default_width_height != (0, 0):
             try:
@@ -51,15 +61,14 @@ def restore_geometry(widget: QWidget, widget_name: str, default_width_height):
             except Exception:
                 logging.warning("Error during restoring default")
 
-
-    logging.debug(f'restore geometry function for {widget_name}')
+    logging.debug(f"restore geometry function for {widget_name}")
     try:
         ini_file_path = pl.Path.home() / pl.Path(".boris")
         if ini_file_path.is_file():
             settings = QSettings(str(ini_file_path), QSettings.IniFormat)
             print(settings.value(f"{widget_name} geometry"))
             widget.restoreGeometry(settings.value(f"{widget_name} geometry"))
-            logging.debug(f'geometry restored for {widget_name}  {settings.value(f"{widget_name} geometry")}')
+            logging.debug(f"geometry restored for {widget_name}  {settings.value(f'{widget_name} geometry')}")
         else:
             default_resize(widget, default_width_height)
     except Exception:
@@ -108,3 +117,20 @@ def set_icons(self, theme_mode: str) -> None:
     self.action_geometric_measurements.setIcon(QIcon(f":/measurement_{theme_mode}"))
     self.actionFind_in_current_obs.setIcon(QIcon(f":/find_{theme_mode}"))
     self.actionExplore_project.setIcon(QIcon(f":/explore_{theme_mode}"))
+
+
+def resize_center(app, window, width: int, height: int) -> None:
+    """
+    resize and center window
+    """
+    window.resize(width, height)
+    screen_geometry = app.primaryScreen().geometry()
+    if window.height() > screen_geometry.height():
+        window.resize(window.width(), int(screen_geometry.height() * 0.8))
+    if window.width() > screen_geometry.width():
+        window.resize(screen_geometry.width(), window.height())
+    # center
+    center_x = (screen_geometry.width() - window.width()) // 2
+    center_y = (screen_geometry.height() - window.height()) // 2
+
+    window.move(center_x, center_y)

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/media_file.py

@@ -20,11 +20,13 @@ This file is part of BORIS.
 
 """
 
+from PySide6.QtWidgets import QFileDialog
+
 from . import config as cfg
 from . import utilities as util
 from . import dialog
 from . import project_functions
-from PySide6.QtWidgets import QFileDialog
+from . import utilities as util
 
 
 def get_info(self) -> None:
@@ -38,17 +40,17 @@ def get_info(self) -> None:
         if "error" in r:
             ffmpeg_output = f"File path: {media_full_path}<br><br>{r['error']}<br><br>"
         else:
-            ffmpeg_output = f"<br><b>{r['analysis_program'] } analysis</b><br>"
+            ffmpeg_output = f"<br><b>{r['analysis_program']} analysis</b><br>"
 
             ffmpeg_output += (
                 f"File path: <b>{media_full_path}</b><br><br>"
                 f"Duration: {r['duration']} seconds ({util.convertTime(self.timeFormat, r['duration'])})<br>"
+                f"FPS: {r['fps']}<br>"
+                f"Resolution: {r['resolution']} pixels<br>"
                 f"Format long name: {r.get('format_long_name', cfg.NA)}<br>"
                 f"Creation time: {r.get('creation_time', cfg.NA)}<br>"
-                f"Resolution: {r['resolution']}<br>"
                 f"Number of frames: {r['frames_number']}<br>"
                 f"Bitrate: {util.smart_size_format(r['bitrate'])}   <br>"
-                f"FPS: {r['fps']}<br>"
                 f"Has video: {r['has_video']}<br>"
                 f"Has audio: {r['has_audio']}<br>"
                 f"File size: {util.smart_size_format(r.get('file size', cfg.NA))}<br>"
@@ -70,6 +72,12 @@ def get_info(self) -> None:
 
             mpv_output = (
                 "<b>MPV information</b><br>"
+                f"Duration: {dw.player.duration} seconds ({util.seconds2time(dw.player.duration)})<br>"
+                # "Position: {} %<br>"
+                f"FPS: {dw.player.container_fps}<br>"
+                # "Rate: {}<br>"
+                f"Resolution: {dw.player.width}x{dw.player.height} pixels<br>"
+                # "Scale: {}<br>"
                 f"Video format: {dw.player.video_format}<br>"
                 # "State: {}<br>"
                 # "Media Resource Location: {}<br>"
@@ -77,12 +85,6 @@ def get_info(self) -> None:
                 # "Track: {}/{}<br>"
                 f"Number of media in media list: {dw.player.playlist_count}<br>"
                 f"Current time position: {dw.player.time_pos}<br>"
-                f"Duration: {dw.player.duration}<br>"
-                # "Position: {} %<br>"
-                f"FPS: {dw.player.container_fps}<br>"
-                # "Rate: {}<br>"
-                f"Video size: {dw.player.width}x{dw.player.height}<br>"
-                # "Scale: {}<br>"
                 f"Aspect ratio: {round(dw.player.width / dw.player.height, 3)}<br>"
                 # "is seekable? {}<br>"
                 # "has_vout? {}<br>"

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/modifier_coding_map_creator.py

@@ -26,8 +26,9 @@ This file is part of BORIS.
 import binascii
 import io
 import json
-import os
+from pathlib import Path
 import re
+import gui_utilities
 
 from PySide6.QtCore import (
     Qt,
@@ -727,7 +728,8 @@ class ModifiersMapCreatorWindow(QMainWindow):
             self.fileName = fn
 
         if self.fileName:
-            if os.path.splitext(self.fileName)[1] != ".boris_map":
+            # if os.path.splitext(self.fileName)[1] != ".boris_map":
+            if Path(self.fileName).suffix != ".boris_map":
                 self.fileName += ".boris_map"
             self.saveMap()
 
@@ -744,7 +746,7 @@ class ModifiersMapCreatorWindow(QMainWindow):
             else:
                 self.fileName = fn
 
-            if self.fileName and os.path.splitext(self.fileName)[1] != ".boris_map":
+            if self.fileName and Path(self.fileName).suffix() != ".boris_map":
                 self.fileName += ".boris_map"
 
         if self.fileName:
@@ -1006,25 +1008,8 @@ if __name__ == "__main__":
 
     app = QApplication(sys.argv)
     window = ModifiersMapCreatorWindow()
-    window.resize(800, 700)
 
-    print(f"{window.width()=}")
-    print(f"{window.height()=}")
-
-    # Get the screen geometry (screen size and position)
-    screen_geometry = app.primaryScreen().geometry()
-
-    print(f"{screen_geometry=}")
-
-    # Calculate the center of the screen
-    center_x = (screen_geometry.width() - window.width()) // 2
-    center_y = (screen_geometry.height() - window.height()) // 2
-
-    print(f"{center_x=}")
-    print(f"{center_y=}")
-
-    # Move the widget to the center of the screen
-    window.move(center_x, center_y)
+    gui_utilities.resize_center(app, window, cfg.CODING_MAP_RESIZE_W, cfg.CODING_MAP_RESIZE_H)
 
     window.show()
     sys.exit(app.exec())

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/observation.py

@@ -1008,27 +1008,30 @@ class Observation(QDialog, Ui_Form):
 
         # check if observation id not empty
         if not self.leObservationId.text():
-            self.qm = QMessageBox()
-            self.qm.setIcon(QMessageBox.Critical)
-            self.qm.setText("The <b>observation id</b> is mandatory and must be unique.")
-            self.qm.exec_()
+            QMessageBox.critical(
+                self,
+                cfg.programName,
+                "The <b>observation id</b> is mandatory and must be unique.",
+            )
             return False
 
         # check if observation_type
         if not any((self.rb_media_files.isChecked(), self.rb_live.isChecked(), self.rb_images.isChecked())):
-            self.qm = QMessageBox()
-            self.qm.setIcon(QMessageBox.Critical)
-            self.qm.setText("Choose an observation type.")
-            self.qm.exec_()
+            QMessageBox.critical(
+                self,
+                cfg.programName,
+                "Choose an observation type.",
+            )
             return False
 
         # check if offset is correct
         if self.cb_time_offset.isChecked():
             if self.obs_time_offset.get_time() is None:
-                self.qm = QMessageBox()
-                self.qm.setIcon(QMessageBox.Critical)
-                self.qm.setText("Check the time offset value.")
-                self.qm.exec_()
+                QMessageBox.critical(
+                    self,
+                    cfg.programName,
+                    "Check the time offset value.",
+                )
                 return False
 
         if self.rb_media_files.isChecked():  # observation based on media file(s)
@@ -1051,18 +1054,20 @@ class Observation(QDialog, Ui_Form):
 
             # check if player #1 is used
             if not players_list or min(players_list) > 1:
-                self.qm = QMessageBox()
-                self.qm.setIcon(QMessageBox.Critical)
-                self.qm.setText("A media file must be loaded in player #1")
-                self.qm.exec_()
+                QMessageBox.critical(
+                    self,
+                    cfg.programName,
+                    "A media file must be loaded in player #1",
+                )
                 return False
 
             # check if players are used in crescent order
             if set(list(range(min(players_list), max(players_list) + 1))) != set(players_list):
-                self.qm = QMessageBox()
-                self.qm.setIcon(QMessageBox.Critical)
-                self.qm.setText("Some player are not used. Please reorganize your media files")
-                self.qm.exec_()
+                QMessageBox.critical(
+                    self,
+                    cfg.programName,
+                    "Some player are not used. Please reorganize your media files",
+                )
                 return False
 
             # check if more media in player #1 and media in other players

{boris_behav_obs-9.3.1 → boris_behav_obs-9.3.3}/boris/observation_operations.py

@@ -888,7 +888,7 @@ def new_observation(self, mode: str = cfg.NEW, obsId: str = "") -> None:
                 self.pj[cfg.OBSERVATIONS][obsId][cfg.CLOSE_BEHAVIORS_BETWEEN_VIDEOS]
             )
 
-    rv = observationWindow.exec_()
+    rv = observationWindow.exec()
 
     # save geometry
     gui_utilities.save_geometry(observationWindow, "new observation")
@@ -2012,7 +2012,7 @@ def initialize_new_media_observation(self) -> bool:
     self.mpv_eof_reached_signal.connect(self.mpv_eof_reached)
     self.video_click_signal.connect(self.player_clicked)
 
-    self.actionPlay.setIcon(QIcon(f":/play_{self.theme_mode()}"))
+    self.actionPlay.setIcon(QIcon(f":/play_{gui_utilities.theme_mode()}"))
 
     self.display_statusbar_info(self.observationId)
 
@@ -2406,15 +2406,15 @@ def event2media_file_name(observation: dict, timestamp: dec) -> Optional[str]:
     """
 
     cumul_media_durations: list = [dec(0)]
-    for media_file in observation[cfg.FILE]["1"]:
+    for media_file in observation[cfg.FILE][cfg.PLAYER1]:
         media_duration = dec(str(observation[cfg.MEDIA_INFO][cfg.LENGTH][media_file]))
-        cumul_media_durations.append(cumul_media_durations[-1] + media_duration)
+        cumul_media_durations.append(round(cumul_media_durations[-1] + media_duration, 3))
 
     cumul_media_durations.remove(dec(0))
 
     # test if timestamp is at end of last media
     if timestamp == cumul_media_durations[-1]:
-        player_idx = len(observation[cfg.FILE]["1"]) - 1
+        player_idx = len(observation[cfg.FILE][cfg.PLAYER1]) - 1
     else:
         player_idx = -1
         for idx, value in enumerate(cumul_media_durations):
@@ -2424,7 +2424,7 @@ def event2media_file_name(observation: dict, timestamp: dec) -> Optional[str]:
                 break
 
     if player_idx != -1:
-        video_file_name = observation[cfg.FILE]["1"][player_idx]
+        video_file_name = observation[cfg.FILE][cfg.PLAYER1][player_idx]
     else:
         video_file_name = None