Updated #TODO priorities

cdl/core/gui/h5io.py

@@ -82,8 +82,6 @@ def __add_object_from_node(self, node: BaseNode) -> None:
         """Add DataLab object from h5 node"""
         obj = node.get_native_object()
         if obj is None:
-            # TODO: Add a warning message to be shown after all imports?
-            # (e.g. "No supported data available in HDF5 file(s)."
             return
         self.uint32_wng = self.uint32_wng or node.uint32_wng
         if isinstance(obj, SignalObj):

cdl/core/gui/objectmodel.py

@@ -45,7 +45,7 @@
     from cdl.core.model.signal import SignalObj
 
 
-# TODO: [P1] Remove this function and use the same mechanism as in reordering methods
+# TODO: [P3] Remove this function and use the same mechanism as in reordering methods
 #
 # This mechanism is as follows (see `reorder_groups` and `reorder_objects` in
 # `ObjectModel` class):

cdl/core/model/base.py

@@ -201,7 +201,7 @@ def config_annotated_shape(
     item.set_style("plot", option)
 
 
-# TODO: Move this function as a method of plot items in PlotPy
+# TODO: [P3] Move this function as a method of plot items in PlotPy
 def set_plot_item_editable(
     item: AbstractShape | AbstractLabelItem | AnnotatedShape, state
 ):

cdl/env.py

@@ -30,7 +30,7 @@ class VerbosityLevels(enum.Enum):
     DEBUG = "debug"
 
 
-# TODO: [P2] Rewrite this class so that options are automatically associated with
+# TODO: [P3] Rewrite this class so that options are automatically associated with
 #       environment variables and command line arguments.
 #
 #       Use the unit test "cdl\tests\backbone\execenv_unit.py" to check that

cdl/tests/features/common/io_app_test.py

@@ -33,7 +33,7 @@ def __testfunc(
         # os.startfile(tmpdir)
         fnames = get_test_fnames(pattern, in_folder)
         execenv.print("    Opening:")
-        # TODO: This test does not support formats that return multiple objects
+        # TODO: [P3] This test does not support formats that return multiple objects
         # (e.g. SIF files with multiple images). As a consequence, it will not test
         # thoroughly the I/O functionalities for these formats (it will keep only the
         # first object in the list of returned objects)

cdl/tests/features/common/resultproperties_app_test.py

@@ -26,7 +26,7 @@ def test_resultproperties():
     """Result properties application test"""
     obj1 = test_data.create_sincos_image()
     with cdltest_app_context(console=False) as win:
-        # TODO: Move the following line outside the with statement and solve
+        # TODO: [P1] Move the following line outside the with statement and solve
         # the "QPixmap: Must construct a QGuiApplication before a QPixmap" error
         obj2 = create_image_with_resultproperties()
         panel = win.signalpanel

wix/makewxs.py

@@ -8,7 +8,7 @@
     Initial version.
 """
 
-# TODO: Localization, eventually.
+# TODO: [P4] Localization, eventually.
 
 import argparse
 import os