From efa3d921aebb429bb09fa79e9d4b370d9677ee0b Mon Sep 17 00:00:00 2001 From: mariosasko Date: Thu, 6 Apr 2023 16:27:27 +0200 Subject: [PATCH] Fix for revision with `/` --- src/huggingface_hub/hf_file_system.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/huggingface_hub/hf_file_system.py b/src/huggingface_hub/hf_file_system.py index e2970d838e..3b464a6ef5 100644 --- a/src/huggingface_hub/hf_file_system.py +++ b/src/huggingface_hub/hf_file_system.py @@ -36,7 +36,7 @@ class HfFileSystemResolvedPath: def unresolve(self): path = ( - f"{REPO_TYPES_URL_PREFIXES.get(self.repo_type, '') + self.repo_id}@{self.revision}/{self.path_in_repo}" + f"{REPO_TYPES_URL_PREFIXES.get(self.repo_type, '') + self.repo_id}@{quote(self.revision, safe='')}/{self.path_in_repo}" .rstrip("/") ) return path @@ -257,7 +257,7 @@ def ls( except EntryNotFoundError: if "/" in resolved_path.path_in_repo: tree_path = self._parent(path) - tree_iter = self._iter_tree(tree_path) + tree_iter = self._iter_tree(tree_path, revision=resolved_path.revision) else: raise else: