[ 
https://issues.apache.org/jira/browse/BEAM-9247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17036348#comment-17036348
 ] 

Ahmet Altay commented on BEAM-9247:
-----------------------------------

I think this will be a confusing API. I will suggest 2 PTransforms of the form
 # input type : Union[text_type, binary_type], vision.types.ImageContext]
 # input type: Union[text_type, binary_type] and a  side input with: 
vision.types.ImageContext (side input could be provided statically or 
dynamically.)

If it is possible to build a unified PTransform with reduced code, offering the 
above two could be done with minimal wrapper code on top of the first.

> [Python] PTransform that integrates Cloud Vision functionality
> --------------------------------------------------------------
>
>                 Key: BEAM-9247
>                 URL: https://issues.apache.org/jira/browse/BEAM-9247
>             Project: Beam
>          Issue Type: Sub-task
>          Components: io-py-gcp
>            Reporter: Kamil Wasilewski
>            Assignee: Elias Djurfeldt
>            Priority: Major
>
> The goal is to create a PTransform that integrates Google Cloud Vision API 
> functionality [1].
> [1] https://cloud.google.com/vision/docs/



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to