From c7b0f62014ecdfb00ed74efaec2fb9f17161af3d Mon Sep 17 00:00:00 2001 From: Joyce Yan <5653616+joyceyan@users.noreply.github.com> Date: Thu, 12 Oct 2023 12:52:06 -0400 Subject: [PATCH] add period for clarity --- cellxgene_schema_cli/cellxgene_schema/validate.py | 2 +- cellxgene_schema_cli/tests/test_schema_compliance.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/cellxgene_schema_cli/cellxgene_schema/validate.py b/cellxgene_schema_cli/cellxgene_schema/validate.py index 6431b7cf8..100004594 100644 --- a/cellxgene_schema_cli/cellxgene_schema/validate.py +++ b/cellxgene_schema_cli/cellxgene_schema/validate.py @@ -755,7 +755,7 @@ def _validate_colors_in_uns_dict(self, uns_dict: dict) -> None: if not obs_unique_values: error_message = f"Colors field uns[{key}] does not have a corresponding categorical field in obs" if column_name in df.columns: - error_message += f" {column_name} is present but is dtype {df[column_name].dtype.name}" + error_message += f". {column_name} is present but is dtype {df[column_name].dtype.name}" self.errors.append(error_message) continue # 2. Verify that the value is a numpy array diff --git a/cellxgene_schema_cli/tests/test_schema_compliance.py b/cellxgene_schema_cli/tests/test_schema_compliance.py index 38181943f..ed6d11dde 100644 --- a/cellxgene_schema_cli/tests/test_schema_compliance.py +++ b/cellxgene_schema_cli/tests/test_schema_compliance.py @@ -1600,7 +1600,7 @@ def test_colors_bool_obs_counterpart(self, validator_with_adata): validator.adata.uns["is_primary_data_colors"] = numpy.array(["green", "purple"]) validator.validate_adata() assert validator.errors == [ - "ERROR: Colors field uns[is_primary_data_colors] does not have a corresponding categorical field in obs is_primary_data is present but is dtype bool" + "ERROR: Colors field uns[is_primary_data_colors] does not have a corresponding categorical field in obs. is_primary_data is present but is dtype bool" ] def test_colors_without_obs_counterpart(self, validator_with_adata):