Update.

PennLINC · Feb 5, 2025 · ff4f5f8 · ff4f5f8
1 parent a318078
commit ff4f5f8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 16 deletions.
diff --git a/cubids/tests/test_cubids.py b/cubids/tests/test_cubids.py
@@ -449,21 +449,6 @@ def _test__get_param_groups(cubids_instance):
     # Add assertions here
 
 
-def _test_round_params(cubids_instance):
-    """Test rounding parameters.
-
-    Parameters
-    ----------
-    cubids_instance : CuBIDS
-        An instance of the CuBIDS class.
-    """
-    param_group_df = pd.DataFrame({"param": [0.123456789]})
-    config = {"param": {"round": 3}}
-    modality = "bold"
-    rounded_params = cubids_instance.round_params(param_group_df, config, modality)
-    # Add assertions here
-
-
 def _test_get_sidecar_metadata(cubids_instance):
     """Test getting sidecar metadata.
 

diff --git a/cubids/utils.py b/cubids/utils.py
@@ -392,8 +392,15 @@ def round_params(df, config, modality):
     Returns
     -------
     pandas.DataFrame
-        DataFrame with the specified columns' values rounded to the requested precision.
+        Modified DataFrame with the specified columns' values rounded to the requested precision.
+
+    Raises
+    ------
+    ValueError
+        If the data type of the column is not supported for rounding, such as strings.
     """
+    df = df.copy()  # don't modify DataFrame in place
+
     to_format = config["sidecar_params"][modality]
     to_format.update(config["derived_params"][modality])