Skip to content

Commit

Permalink
Add root_url to DaskMSStore. Fixes incorrect stores for s3.
Browse files Browse the repository at this point in the history
  • Loading branch information
JSKenyon committed Sep 14, 2023
1 parent f0c96f0 commit 4c7b856
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion daskms/experimental/fragments/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ def get_ancestry(store, only_required=True):
store = DaskMSStore(store)

while True:
root_store = DaskMSStore(store.root)
root_store = DaskMSStore(store.root_url)

if store.exists():
try:
Expand Down
4 changes: 4 additions & 0 deletions daskms/fsspec_store.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,10 @@ def assert_type(self, store_type):
def url(self):
return f"{self.fs.unstrip_protocol(self.canonical_path)}"

@property
def root_url(self):
return f"{self.fs.unstrip_protocol(self.root)}"

def subdirectories(self):
return [
d["name"]
Expand Down

0 comments on commit 4c7b856

Please sign in to comment.