Fix: update file loading behavior to ignore errors when loading unsupported files

Fix #164

Author: PierreRaybaut

CHANGELOG.md

@@ -19,6 +19,7 @@ See DataLab [roadmap page](https://datalab-platform.com/en/contributing/roadmap.
 * Fixed [Issue #161](https://github.com/DataLab-Platform/DataLab/issues/161) - Refresh image items only if necessary (when editing ROI, pasting/deleting metadata)
 * Fixed [Issue #162](https://github.com/DataLab-Platform/DataLab/issues/162) - View in a new window: when displaying multiple images, the item list panel should be visible
 * Fixed [Issue #163](https://github.com/DataLab-Platform/DataLab/issues/163) - Open from directory: expected one group per folder when loading multiple files
+* Fixed [Issue #164](https://github.com/DataLab-Platform/DataLab/issues/164) - Open from directory: unsupported files should be ignored when loading files recursively, to avoid warning popup dialog boxes
 
 ## DataLab Version 0.19.0 ##
 

cdl/core/gui/panel/base.py

@@ -919,7 +919,7 @@ def load_from_directory(self, directory: str | None = None) -> list[TypeObj]:
                     if osp.isfile(osp.join(path, fname))
                 ]
                 new_objs = self.load_from_files(
-                    fnames, create_group=False, ignore_unknown=True
+                    fnames, create_group=False, ignore_errors=True
                 )
                 objs += new_objs
                 if len(new_objs) == 0:
@@ -930,7 +930,7 @@ def load_from_files(
         self,
         filenames: list[str] | None = None,
         create_group: bool = False,
-        ignore_unknown: bool = False,
+        ignore_errors: bool = False,
     ) -> list[TypeObj]:
         """Open objects from file (signals/images), add them to DataLab and return them.
 
@@ -939,7 +939,7 @@ def load_from_files(
             create_group: if True, create a new group if more than one object is loaded
              for a single file. Defaults to False: all objects are added to the current
              group.
-            ignore_unknown: if True, ignore unknown file types (default: False)
+            ignore_errors: if True, ignore errors when loading files. Defaults to False.
 
         Returns:
             list of new objects
@@ -957,8 +957,8 @@ def load_from_files(
                 Conf.main.base_dir.set(filename)
                 try:
                     objs += self.__load_from_file(filename, create_group=create_group)
-                except NotImplementedError as exc:
-                    if ignore_unknown:
+                except Exception as exc:  # pylint: disable=broad-except
+                    if ignore_errors:
                         # Ignore unknown file types
                         pass
                     else:

doc/locale/fr/LC_MESSAGES/api/core.gui/panel.po

@@ -8,7 +8,7 @@ msgid ""
 msgstr ""
 "Project-Id-Version: DataLab \n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-04-08 10:13+0200\n"
+"POT-Creation-Date: 2025-04-08 10:22+0200\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language: fr\n"
@@ -346,7 +346,7 @@ msgstr ""
 msgid "if True, create a new group if more than one object is loaded for a single file. Defaults to False: all objects are added to the current group."
 msgstr ""
 
-msgid "if True, ignore unknown file types (default: False)"
+msgid "if True, ignore errors when loading files. Defaults to False."
 msgstr ""
 
 msgid "Save selected objects to files (signal/image)."

doc/locale/fr/LC_MESSAGES/contributing/changelog.po

@@ -7,7 +7,7 @@ msgid ""
 msgstr ""
 "Project-Id-Version: DataLab \n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2025-04-08 10:13+0200\n"
+"POT-Creation-Date: 2025-04-08 10:22+0200\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language: fr\n"
@@ -69,6 +69,10 @@ msgstr "Correction de l'[Issue #162](https://github.com/DataLab-Platform/DataLab
 msgid "Fixed [Issue #163](https://github.com/DataLab-Platform/DataLab/issues/163) - Open from directory: expected one group per folder when loading multiple files"
 msgstr "Ceci corrige l'[Issue #163](https://github.com/DataLab-Platform/DataLab/issues/163) - Ouvrir depuis le répertoire : un groupe par dossier attendu lors du chargement de plusieurs fichiers"
 
+#, fuzzy
+msgid "Fixed [Issue #164](https://github.com/DataLab-Platform/DataLab/issues/164) - Open from directory: unsupported files should be ignored when loading files recursively, to avoid warning popup dialog boxes"
+msgstr "Ceci corrige l'[Issue #163](https://github.com/DataLab-Platform/DataLab/issues/163) - Ouvrir depuis le répertoire : un groupe par dossier attendu lors du chargement de plusieurs fichiers"
+
 msgid "DataLab Version 0.19.0"
 msgstr "DataLab Version 0.19.0"
 
@@ -2208,3 +2212,4 @@ msgstr "Symétrie (verticale/horizontale)"
 
 msgid "Console: added configurable external editor (default: VSCode) to follow the traceback links to the source code"
 msgstr "Console : ajout d'un éditeur externe configurable (par défaut : VSCode) pour suivre les liens de poursuite vers le code source"
+