Skip to content

Commit 9634268

Browse files
zhu0619jstlaurent
andauthored
Fix/Update polarisfs.base_path woth version number (#189)
* update base_path * remove duplicated exception * Update polaris/hub/polarisfs.py Remove leftover debug logs. Co-authored-by: Julien St-Laurent <[email protected]> * restore exception --------- Co-authored-by: Julien St-Laurent <[email protected]>
1 parent d05937e commit 9634268

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

polaris/hub/client.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -404,6 +404,7 @@ def open_zarr_file(
404404
except HTTPStatusError as error:
405405
# In this case, we can pass the response to provide more information
406406
raise PolarisHubError(message="Error opening Zarr store", response=error.response) from error
407+
# This catches all other types of exceptions
407408
except Exception as error:
408409
raise PolarisHubError(message="Error opening Zarr store") from error
409410

polaris/hub/polarisfs.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,8 @@ def __init__(
5454

5555
# Prefix to remove from ls entries
5656
self.prefix = f"dataset/{dataset_owner}/{sluggify(dataset_name)}/"
57-
self.base_path = f"/storage/{self.prefix.rstrip('/')}"
57+
# Base path for uploading. Please pay attention on path version update.
58+
self.base_path = f"/v1/storage/{self.prefix.rstrip('/')}"
5859

5960
@staticmethod
6061
def is_polarisfs_path(path: str) -> bool:

0 commit comments

Comments
 (0)