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/09/02 13:33:38 UTC

[GitHub] [beam] AnandInguva commented on pull request #21738: Inference benchmark tests

AnandInguva commented on PR #21738:
URL: https://github.com/apache/beam/pull/21738#issuecomment-1235512717

   >> I'm also a little nervous about this change - could you help me understand why we need it?
   
   The `LoadTestOptions` has an arg named `input_options`. RunInference perf tests use LoadTest as the base class since LoadTest class doc string instructs to use it for perf tests. RunInference perf tests need an option named `input` but if I provide that, then the `argparse` maps `input` to `input_options`. 
   
   More at https://docs.python.org/3/library/argparse.html#allow-abbrev


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