Skip to content

Commit

Permalink
Upgrade fsspec and s3fs versions
Browse files Browse the repository at this point in the history
Summary: Need to update code for fsspec/filesystem_spec#1394.

Reviewed By: stepanhruda

Differential Revision: D50388544

fbshipit-source-id: 49944758ac7c87e828950d997ad1d6a386b125d2
  • Loading branch information
alanhdu authored and facebook-github-bot committed Oct 25, 2023
1 parent 1fe0607 commit a224719
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 14 deletions.
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.9.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
26 changes: 14 additions & 12 deletions torchx/workspace/test/dir_workspace_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,23 +90,25 @@ def test_torchxignore(self) -> None:

_copy_to_dir("memory://torchxignore", "memory://torchxignoredest")

files = fs.glob("torchxignoredest/*") + fs.glob("torchxignoredest/**/*")
files = fs.glob("torchxignoredest/**/*")
# strip prefix
files = [
os.path.normpath(file.partition("torchxignoredest/")[2]) for file in files
]
print(files)
self.assertCountEqual(
files,
{
".torchxignore",
"dir",
"dir/file",
"dir/ignorefile",
"foo.sh",
"ignorefilesuffix",
"unignore",
},
self.assertListEqual(
sorted(files),
sorted(
[
".torchxignore",
"dir",
"dir/file",
"dir/ignorefile",
"foo.sh",
"ignorefilesuffix",
"unignore",
]
),
)


Expand Down

0 comments on commit a224719

Please sign in to comment.