Fix some comments [skip CI]

gem · Dec 16, 2019 · 5163a33 · 5163a33
1 parent be7ecb8
commit 5163a33
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/svir/dialogs/load_dmg_by_asset_as_layer_dialog.py b/svir/dialogs/load_dmg_by_asset_as_layer_dialog.py
@@ -174,7 +174,7 @@ def get_field_types(self, **kwargs):
             self.default_field_name = "%s_%s_mean" % (
                 self.loss_type_cbx.currentText(),
                 self.dmg_state_cbx.currentText())
-        # FIXME: check if it's ok to have all numeric fields
+        # NOTE: assuming that all fields are numeric
         field_types = {field_name: 'F' for field_name in field_names}
         return field_types
 

diff --git a/svir/dialogs/load_hcurves_as_layer_dialog.py b/svir/dialogs/load_hcurves_as_layer_dialog.py
@@ -108,7 +108,8 @@ def get_field_types(self, **kwargs):
                     continue
                 for iml in self.dataset[rlz_or_stat][imt].dtype.names:
                     field_name = "%s_%s_%s" % (rlz_or_stat, imt, iml)
-                    field_types[field_name] = 'F'  # FIXME: check if type is ok
+                    # NOTE: assuming that all fields are numeric
+                    field_types[field_name] = 'F'
         return field_types
 
     def on_iml_changed(self):

diff --git a/svir/dialogs/load_uhs_as_layer_dialog.py b/svir/dialogs/load_uhs_as_layer_dialog.py
@@ -102,7 +102,7 @@ def get_field_types(self, **kwargs):
             field_names.extend([
                 "%s_%s" % (rlz_or_stat, imt)
                 for imt in self.dataset[rlz_or_stat][poe].dtype.names])
-        # FIXME: check that all fields are numeric
+        # NOTE: assuming that all fields are numeric
         field_types = {field_name: 'F' for field_name in field_names}
         return field_types