Skip to content

Commit

Permalink
add 'tabulate' to the requirements.txt since the CLI cmd_list uses it (
Browse files Browse the repository at this point in the history
…#627)

Summary:
Fixes a bug introduced in https://github.com/pytorch/torchx/pull/579/files#diff-0aff65b8fe78257a723972926d63b801c495e54669248db41bed7536055aad92

Using `tabulate` in `cmd_list.py` but not adding it to `requirements.txt` causes torchx CLI to fail:

```
$ torchx runopts aws_batch

Traceback (most recent call last):
  File "/home/ubuntu/.pyenv/versions/venv39/bin/torchx", line 5, in <module>
    from torchx.cli.main import main
  File "/home/ubuntu/.pyenv/versions/3.9.13/envs/venv39/lib/python3.9/site-packages/torchx/cli/main.py", line 17, in <module>
    from torchx.cli.cmd_list import CmdList
  File "/home/ubuntu/.pyenv/versions/3.9.13/envs/venv39/lib/python3.9/site-packages/torchx/cli/cmd_list.py", line 11, in <module>
    from tabulate import tabulate
ModuleNotFoundError: No module named 'tabulate'
```

Pull Request resolved: #627

Reviewed By: priyaramani

Differential Revision: D40742523

Pulled By: d4l3k

fbshipit-source-id: 092bfce9ee6857806a84b9d6b5f0250151413b72
  • Loading branch information
kiukA9 authored and facebook-github-bot committed Oct 27, 2022
1 parent f61f22c commit 197bc21
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,5 @@ pyyaml
docker
filelock
fsspec
tabulate

0 comments on commit 197bc21

Please sign in to comment.