Re: [PR] [YAML] Fraud Detection MLOps Workflow Example [beam]

["charlespnh (via GitHub)" <gi...@apache.org>]

charlespnh commented on code in PR #36012:
URL: https://github.com/apache/beam/pull/36012#discussion_r2314612774


##########
sdks/python/apache_beam/yaml/examples/transforms/ml/fraud_detection/fraud_detection_mlops_beam_yaml_sdk.ipynb:
##########
@@ -0,0 +1,1329 @@
+{
+ "cells": [
+  {
+   "cell_type": "code",
+   "source": [
+    "# @title ###### Licensed to the Apache Software Foundation (ASF), Version 2.0 (the \"License\")\n",

Review Comment:
   I've given a lot of thought about this: The workflow is pretty complex with model training and several pipelines and custom transforms. I thought it makes more sense to place all these parts in a notebook so people can follow the example easily.
   Otherwise if the pipelines and model training are in separate files then a bash script is needed to tie them together into a workflow, and it wouldn't have a nice flow of explanation that a notebook can have.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org