[
https://issues.apache.org/jira/browse/BEAM-9247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17036987#comment-17036987
]
Elias Djurfeldt edited comment on BEAM-9247 at 2/17/20 12:04 PM:
-----------------------------------------------------------------
Think I only got one question before I can implement this:
Are we expecting the side input to be AsIter, where the side input just
specifies an iterable of video contexts.
{code:java}
('context1', 'context2', ...){code}
Or are we expecting a mapping (AsDict) on the form:
{code:java}
{'gs://cloud-samples-data/video/cat.mp4': 'context1',
'gs://other-video/sample.mp4': 'context2', ...}
{code}
AsDict makes the mapping easy in the PTransform but forces materialisation of
the entire side input. What's your take [~altay]?
Another alternative is of course having a tuple mapping AsIter side input
{code:java}
('gs://cloud-samples-data/video/cat.mp4'`, 'context1'){code}
Lets us use the benefits of generators but will still need to loop through it
for each element if I'm understanding side inputs correctly.
was (Author: eliasdjur):
Think I only got one question before I can implement this:
Are we expecting the side input to be AsIter, where the side input just
specifies an iterable of video contexts.
{code:java}
('context1', 'context2', ...){code}
Or are we expecting a mapping (AsDict) on the form:
{code:java}
{'gs://cloud-samples-data/video/cat.mp4': 'context1',
'gs://other-video/sample.mp4': 'context2', ...}
{code}
> [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)
