kerrydc commented on PR #26553: URL: https://github.com/apache/beam/pull/26553#issuecomment-1536791704
> LGTM. One request: Can we add a test for this? > > Also, do we want to add documentation at https://beam.apache.org/documentation/sdks/python-machine-learning/ for passing keyword args and what keyword args do we accept or `pydocs` are sufficient? I will let @damccorm chime in for this Added tests, thanks! -- 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]
