damccorm commented on code in PR #28777:
URL: https://github.com/apache/beam/pull/28777#discussion_r1346314366
##########
examples/notebooks/beam-ml/automatic_model_refresh.ipynb:
##########
@@ -132,8 +119,6 @@
"from typing import Tuple\n",
"\n",
"import apache_beam as beam\n",
- "from apache_beam.examples.inference.tensorflow_imagenet_segmentation
import PostProcessor\n",
- "from apache_beam.examples.inference.tensorflow_imagenet_segmentation
import read_image\n",
Review Comment:
Right now, read_image is still called, but its not defined anywhere.
Originally we imported it, but that's fragile since the example could
change/doesn't have rigorous API guarantees. We should define the function next
to its usage instead
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
