From ea4a0be303cb665f55f4aaf195b5bbdc09a4d578 Mon Sep 17 00:00:00 2001 From: Yohanna Lisnichuk Date: Sat, 10 Feb 2024 14:36:46 -0300 Subject: [PATCH] lint: define custom exception class --- component_charts.ipynb | 5 ++++- template_data_quality_feedback.ipynb | 5 ++++- template_publisher_analysis.ipynb | 5 ++++- template_structure_and_format_feedback.ipynb | 5 ++++- template_usability_checks.ipynb | 5 ++++- template_usability_checks_fieldlist.ipynb | 5 ++++- template_usability_checks_registry.ipynb | 5 ++++- 7 files changed, 28 insertions(+), 7 deletions(-) diff --git a/component_charts.ipynb b/component_charts.ipynb index 656fa1c..0b76031 100644 --- a/component_charts.ipynb +++ b/component_charts.ipynb @@ -56,11 +56,14 @@ "source": [ "import altair as alt\n", "\n", + "class MissingColumns(Exception):\n", + " def __init__(self, columns):\n", + " super().__init__(f\"The source data is missing one or more of these columns: {columns}\")\n", "\n", "def check_columns(columns, data):\n", " # check if input contains the right columns\n", " if not columns.issubset(data.columns):\n", - " raise ValueError(f\"Input data must contain the columns {columns}\")\n", + " raise MissingColumns(columns)\n", "\n", "\n", "def plot_release_count(release_counts):\n", diff --git a/template_data_quality_feedback.ipynb b/template_data_quality_feedback.ipynb index 1c373ff..886e968 100644 --- a/template_data_quality_feedback.ipynb +++ b/template_data_quality_feedback.ipynb @@ -176,11 +176,14 @@ "source": [ "import altair as alt\n", "\n", + "class MissingColumns(Exception):\n", + " def __init__(self, columns):\n", + " super().__init__(f\"The source data is missing one or more of these columns: {columns}\")\n", "\n", "def check_columns(columns, data):\n", " # check if input contains the right columns\n", " if not columns.issubset(data.columns):\n", - " raise ValueError(f\"Input data must contain the columns {columns}\")\n", + " raise MissingColumns(columns)\n", "\n", "\n", "def plot_release_count(release_counts):\n", diff --git a/template_publisher_analysis.ipynb b/template_publisher_analysis.ipynb index e92233d..3838d17 100644 --- a/template_publisher_analysis.ipynb +++ b/template_publisher_analysis.ipynb @@ -176,11 +176,14 @@ "source": [ "import altair as alt\n", "\n", + "class MissingColumns(Exception):\n", + " def __init__(self, columns):\n", + " super().__init__(f\"The source data is missing one or more of these columns: {columns}\")\n", "\n", "def check_columns(columns, data):\n", " # check if input contains the right columns\n", " if not columns.issubset(data.columns):\n", - " raise ValueError(f\"Input data must contain the columns {columns}\")\n", + " raise MissingColumns(columns)\n", "\n", "\n", "def plot_release_count(release_counts):\n", diff --git a/template_structure_and_format_feedback.ipynb b/template_structure_and_format_feedback.ipynb index c65be0a..20f42a9 100644 --- a/template_structure_and_format_feedback.ipynb +++ b/template_structure_and_format_feedback.ipynb @@ -176,11 +176,14 @@ "source": [ "import altair as alt\n", "\n", + "class MissingColumns(Exception):\n", + " def __init__(self, columns):\n", + " super().__init__(f\"The source data is missing one or more of these columns: {columns}\")\n", "\n", "def check_columns(columns, data):\n", " # check if input contains the right columns\n", " if not columns.issubset(data.columns):\n", - " raise ValueError(f\"Input data must contain the columns {columns}\")\n", + " raise MissingColumns(columns)\n", "\n", "\n", "def plot_release_count(release_counts):\n", diff --git a/template_usability_checks.ipynb b/template_usability_checks.ipynb index 927d055..a83342f 100644 --- a/template_usability_checks.ipynb +++ b/template_usability_checks.ipynb @@ -176,11 +176,14 @@ "source": [ "import altair as alt\n", "\n", + "class MissingColumns(Exception):\n", + " def __init__(self, columns):\n", + " super().__init__(f\"The source data is missing one or more of these columns: {columns}\")\n", "\n", "def check_columns(columns, data):\n", " # check if input contains the right columns\n", " if not columns.issubset(data.columns):\n", - " raise ValueError(f\"Input data must contain the columns {columns}\")\n", + " raise MissingColumns(columns)\n", "\n", "\n", "def plot_release_count(release_counts):\n", diff --git a/template_usability_checks_fieldlist.ipynb b/template_usability_checks_fieldlist.ipynb index 94282ff..38f1caf 100644 --- a/template_usability_checks_fieldlist.ipynb +++ b/template_usability_checks_fieldlist.ipynb @@ -176,11 +176,14 @@ "source": [ "import altair as alt\n", "\n", + "class MissingColumns(Exception):\n", + " def __init__(self, columns):\n", + " super().__init__(f\"The source data is missing one or more of these columns: {columns}\")\n", "\n", "def check_columns(columns, data):\n", " # check if input contains the right columns\n", " if not columns.issubset(data.columns):\n", - " raise ValueError(f\"Input data must contain the columns {columns}\")\n", + " raise MissingColumns(columns)\n", "\n", "\n", "def plot_release_count(release_counts):\n", diff --git a/template_usability_checks_registry.ipynb b/template_usability_checks_registry.ipynb index 3678340..817a5fb 100644 --- a/template_usability_checks_registry.ipynb +++ b/template_usability_checks_registry.ipynb @@ -176,11 +176,14 @@ "source": [ "import altair as alt\n", "\n", + "class MissingColumns(Exception):\n", + " def __init__(self, columns):\n", + " super().__init__(f\"The source data is missing one or more of these columns: {columns}\")\n", "\n", "def check_columns(columns, data):\n", " # check if input contains the right columns\n", " if not columns.issubset(data.columns):\n", - " raise ValueError(f\"Input data must contain the columns {columns}\")\n", + " raise MissingColumns(columns)\n", "\n", "\n", "def plot_release_count(release_counts):\n",