Merge pull request #29 from comet-toolkit/small_fixes

shunt16 · web-flow · commit 91c343c8365c · 2024-04-02T15:10:08.000+01:00
fix bug when adding component with templater that already exists
diff --git a/obsarray/unc_accessor.py b/obsarray/unc_accessor.py
@@ -877,7 +877,8 @@ def _add_unc_var(
                     self._obj[obs_var].attrs["unc_comps"]
                 ]
 
-            self._obj[obs_var].attrs["unc_comps"].append(unc_var)
+            if unc_var not in self._obj[obs_var].attrs["unc_comps"]:
+                self._obj[obs_var].attrs["unc_comps"].append(unc_var)
 
         else:
             self._obj[obs_var].attrs["unc_comps"] = unc_var

Original file line number	Diff line number	Diff line change
`@@ -877,7 +877,8 @@ def _add_unc_var(`
`877`	`877`	`self._obj[obs_var].attrs["unc_comps"]`
`878`	`878`	`]`
`879`	`879`
`880`		`- self._obj[obs_var].attrs["unc_comps"].append(unc_var)`
	`880`	`+ if unc_var not in self._obj[obs_var].attrs["unc_comps"]:`
	`881`	`+ self._obj[obs_var].attrs["unc_comps"].append(unc_var)`
`881`	`882`
`882`	`883`	`else:`
`883`	`884`	`self._obj[obs_var].attrs["unc_comps"] = unc_var`