You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/02/07 15:52:30 UTC

[GitHub] [beam] ryanthompson591 commented on a change in pull request #16517: BEAM-13439 Type annotation for ptransform_fn

ryanthompson591 commented on a change in pull request #16517:
URL: https://github.com/apache/beam/pull/16517#discussion_r800797960



##########
File path: sdks/python/apache_beam/transforms/ptransform.py
##########
@@ -1003,6 +1003,8 @@ def default_label(self):
 
 
 def ptransform_fn(fn):
+  # type: (Callable) -> (...) -> _PTransformFnPTransform

Review comment:
       I'm trying to understand the format here.
   
   I thought that the format was:
   type: input args -> return type
   
   But here you have  an extra chunk of arguments in between, is that supposed to signify the args or the callable?




-- 
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