From 2cbcbec04dcf6ab3fabacd5ba7e60f43cc0fe502 Mon Sep 17 00:00:00 2001 From: Code Monkey Date: Sun, 5 Dec 2021 21:46:16 +0000 Subject: [PATCH] Update TODO comments --- .../graphviz/visualizers/pipeline_run_dag_visualizer.py | 2 +- .../plotly/visualizers/pipeline_lineage_visualizer.py | 2 +- src/zenml/metadata/base_metadata_store.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/zenml/integrations/graphviz/visualizers/pipeline_run_dag_visualizer.py b/src/zenml/integrations/graphviz/visualizers/pipeline_run_dag_visualizer.py index dea9aa98e95..4fab342b806 100644 --- a/src/zenml/integrations/graphviz/visualizers/pipeline_run_dag_visualizer.py +++ b/src/zenml/integrations/graphviz/visualizers/pipeline_run_dag_visualizer.py @@ -26,7 +26,7 @@ logger = get_logger(__name__) -# TODO [MEDIUM]: This integration is working but not really doing much. We +# TODO [ENG-220]: This integration is working but not really doing much. We # should use graphviz to show the pipeline perhaps (not the run). diff --git a/src/zenml/integrations/plotly/visualizers/pipeline_lineage_visualizer.py b/src/zenml/integrations/plotly/visualizers/pipeline_lineage_visualizer.py index d33ff0f563b..7e9b0fa5003 100644 --- a/src/zenml/integrations/plotly/visualizers/pipeline_lineage_visualizer.py +++ b/src/zenml/integrations/plotly/visualizers/pipeline_lineage_visualizer.py @@ -25,7 +25,7 @@ logger = get_logger(__name__) -# TODO [MEDIUM]: This integration is working but not really doing much. We +# TODO [ENG-221]: This integration is working but not really doing much. We # should use plotly in more useful ways. diff --git a/src/zenml/metadata/base_metadata_store.py b/src/zenml/metadata/base_metadata_store.py index 4c9a3ae2616..e7e6d051e4f 100644 --- a/src/zenml/metadata/base_metadata_store.py +++ b/src/zenml/metadata/base_metadata_store.py @@ -121,7 +121,7 @@ def _get_step_view_from_execution( if not k.startswith(INTERNAL_EXECUTION_PARAMETER_PREFIX) } - # TODO [MEDIUM]: This is a lot of querying to the metadata store. We + # TODO [ENG-222]: This is a lot of querying to the metadata store. We # should refactor and make it nicer. Probably it makes more sense # to first get `executions_ids_for_current_run` and then filter on # `event.execution_id in execution_ids_for_current_run`.