Merge pull request #440 from alercebroker/fix/remove_workflows_produc…

…tion_ci removed print. removed elasstic models build
alercebroker · May 28, 2024 · 70bf0f3 · 70bf0f3
2 parents 86c29ad + 0764790
commit 70bf0f3
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 33 deletions.
diff --git a/.github/workflows/production_ci.yaml b/.github/workflows/production_ci.yaml
@@ -77,35 +77,35 @@ jobs:
     secrets:
       GH_TOKEN: ${{ secrets.ADMIN_TOKEN }}
 
-  build-lc-classification-balto-dagger:
-    needs: update_versions
-    uses: ./.github/workflows/template_build_with_dagger.yaml
-    with:
-      stage: production
-      extra-args: lc_classification_step_balto --build-args=model:balto --package-dir=lc_classification_step
-      submodules: true
-    secrets:
-      GH_TOKEN: ${{ secrets.ADMIN_TOKEN }}
+  # build-lc-classification-balto-dagger:
+  #   needs: update_versions
+  #   uses: ./.github/workflows/template_build_with_dagger.yaml
+  #   with:
+  #     stage: production
+  #     extra-args: lc_classification_step_balto --build-args=model:balto --package-dir=lc_classification_step
+  #     submodules: true
+  #   secrets:
+  #     GH_TOKEN: ${{ secrets.ADMIN_TOKEN }}
 
-  build-lc-classification-messi-dagger:
-    needs: update_versions
-    uses: ./.github/workflows/template_build_with_dagger.yaml
-    with:
-      stage: production
-      extra-args: lc_classification_step_messi --build-args=model:messi --package-dir=lc_classification_step
-      submodules: true
-    secrets:
-      GH_TOKEN: ${{ secrets.ADMIN_TOKEN }}
+  # build-lc-classification-messi-dagger:
+  #   needs: update_versions
+  #   uses: ./.github/workflows/template_build_with_dagger.yaml
+  #   with:
+  #     stage: production
+  #     extra-args: lc_classification_step_messi --build-args=model:messi --package-dir=lc_classification_step
+  #     submodules: true
+  #   secrets:
+  #     GH_TOKEN: ${{ secrets.ADMIN_TOKEN }}
 
-  build-lc-classification-mlp-dagger:
-    needs: update_versions
-    uses: ./.github/workflows/template_build_with_dagger.yaml
-    with:
-      stage: production
-      extra-args: lc_classification_step_mlp --build-args=model:mlp --package-dir=lc_classification_step
-      submodules: true
-    secrets:
-      GH_TOKEN: ${{ secrets.ADMIN_TOKEN }}
+  # build-lc-classification-mlp-dagger:
+  #   needs: update_versions
+  #   uses: ./.github/workflows/template_build_with_dagger.yaml
+  #   with:
+  #     stage: production
+  #     extra-args: lc_classification_step_mlp --build-args=model:mlp --package-dir=lc_classification_step
+  #     submodules: true
+  #   secrets:
+  #     GH_TOKEN: ${{ secrets.ADMIN_TOKEN }}
 
   build-lc-classification-ztf-dagger:
     needs: update_versions
@@ -214,9 +214,9 @@ jobs:
       - build-correction-dagger
       - build-early-classification-dagger
       - build-feature-dagger
-      - build-lc-classification-balto-dagger
-      - build-lc-classification-messi-dagger
-      - build-lc-classification-mlp-dagger
+      # - build-lc-classification-balto-dagger
+      # - build-lc-classification-messi-dagger
+      # - build-lc-classification-mlp-dagger
       - build-lc-classification-ztf-dagger
       - build-lightcurve-dagger
       - build-magstats-dagger

diff --git a/lc_classification_step/lc_classification/core/step.py b/lc_classification_step/lc_classification/core/step.py
@@ -180,9 +180,6 @@ def post_execute(self, result: Tuple[OutputDTO, List[dict], DataFrame]):
             classifier_version=self.classifier_version,
         )
         self.produce_scribe(parsed_result.value)
-        print(
-            f"---SCRIBE PARSED---\n\n{json.dumps(parsed_result.value)}\n\n--------"
-        )
         return result
 
     def tear_down(self):