Skip to content

Commit 26fbb50

Browse files
committed
consistent naming for astra paths so SDSSPath doesnt think we need cat_id and catalogid
1 parent 97da5b5 commit 26fbb50

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

python/sdss_access/path/path.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -336,8 +336,8 @@ def extract(self, name, example):
336336
template = re.sub('@plateid6[|]', '{plateid:0>6}', template)
337337
if re.search('@component_default[|]', template):
338338
template = re.sub('@component_default[|]', '{component_default}', template)
339-
if re.search('@catalogid_groups[|]', template):
340-
template = re.sub('@catalogid_groups[|]', '{catalogid_groups}', template)
339+
if re.search('@cat_id_groups[|]', template):
340+
template = re.sub('@cat_id_groups[|]', '{cat_id_groups}', template)
341341

342342
# check if template has any brackets
343343
haskwargs = re.search('[{}]', template)
@@ -1219,7 +1219,7 @@ def healpixgrp(self, filetype, **kwargs):
12191219
subdir = "{:d}".format(healpix // 1000)
12201220
return subdir
12211221

1222-
def catalogid_groups(self, filetype, **kwargs):
1222+
def cat_id_groups(self, filetype, **kwargs):
12231223
'''
12241224
Return a folder structure to group data together based on their catalog
12251225
identifier so that we don't have too many files in any one folder.
@@ -1228,7 +1228,7 @@ def catalogid_groups(self, filetype, **kwargs):
12281228
----------
12291229
filetype : str
12301230
File type parameter.
1231-
catalogid : int or str
1231+
cat_id : int or str
12321232
SDSS-V catalog identifier
12331233
12341234
Returns
@@ -1238,8 +1238,8 @@ def catalogid_groups(self, filetype, **kwargs):
12381238
'''
12391239
# with k = 100 then even with 10 M sources, each folder will have ~1,000 files
12401240
k = 100
1241-
catalogid = int(kwargs['catalogid'])
1242-
return f"{(catalogid // k) % k:0>2.0f}/{catalogid % k:0>2.0f}"
1241+
cat_id = int(kwargs['cat_id'])
1242+
return f"{(cat_id // k) % k:0>2.0f}/{cat_id % k:0>2.0f}"
12431243

12441244
def component_default(self, filetype, **kwargs):
12451245
''' Return the component name, if given.

0 commit comments

Comments
 (0)