rszper commented on code in PR #22069: URL: https://github.com/apache/beam/pull/22069#discussion_r908946773
########## sdks/python/apache_beam/examples/inference/README.md: ########## @@ -37,17 +37,18 @@ The RunInference API supports the PyTorch framework. To use PyTorch locally, fir pip install torch==1.11.0 ``` -If you are using pretrained models from Pytorch's `torchvision.models` [subpackage](https://pytorch.org/vision/0.12/models.html#models-and-pre-trained-weights), you may also need to install `torchvision`. +If you are using pretrained models from Pytorch's `torchvision.models` [subpackage](https://pytorch.org/vision/0.12/models.html#models-and-pre-trained-weights), you might also need to install `torchvision`. Review Comment: If they need the dependencies for this example to work, we should say, "..., you must also install `torchvision`." or "..., you must also have `torchvision` installed." We can definitely add a line saying that these installation requirements are for the example files provided on this page. -- 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]
