Skip to content

Commit

Permalink
Update torchx for fsspec 2023.10 (#784)
Browse files Browse the repository at this point in the history
  • Loading branch information
alanhdu committed Nov 2, 2023
1 parent c038626 commit fb819df
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
2 changes: 1 addition & 1 deletion dev-requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ boto3
captum>=0.4.0
docker
flake8==3.9.0
fsspec[s3]==2023.1.0
fsspec[s3]==2023.10.0
google-api-core
google-cloud-batch>=0.5.0
google-cloud-logging>=3.0.0
Expand Down
3 changes: 1 addition & 2 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@ importlib-metadata
pyyaml
docker
filelock
# more recent versions of fsspec causes torchx/workspace/test/dir_workspace_test#test_torchcxignore to fail
fsspec==2023.1.0
fsspec==2023.10.0
# To resolve confliciting dependencies for urllib3: https://github.com/pytorch/torchx/actions/runs/3484190429/jobs/5828784263#step:4:552
urllib3<1.27,>=1.21.1
tabulate
7 changes: 3 additions & 4 deletions torchx/workspace/test/dir_workspace_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,11 +92,10 @@ def test_torchxignore(self) -> None:

files = fs.glob("torchxignoredest/*") + fs.glob("torchxignoredest/**/*")
# strip prefix
files = [
files = {
os.path.normpath(file.partition("torchxignoredest/")[2]) for file in files
]
print(files)
self.assertCountEqual(
}
self.assertSetEqual(
files,
{
".torchxignore",
Expand Down

0 comments on commit fb819df

Please sign in to comment.