Skip to content

Commit a9423d8

Browse files
Jorge Fernandez Hernandezbsipocz
authored andcommitted
EUCLIDMNGT-1275 rename parameter dataset to dataset_release
1 parent e287268 commit a9423d8

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

astroquery/esa/euclid/core.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1278,8 +1278,7 @@ def get_datalinks(self, ids, *, linking_parameter='SOURCE_ID', verbose=False):
12781278
return self.__eucliddata.get_datalinks(ids=ids, linking_parameter=linking_parameter, verbose=verbose)
12791279

12801280
def get_scientific_data_product_list(self, *, observation_id=None, tile_index=None, category=None, group=None,
1281-
product_type=None,
1282-
release='LEVEL_3_RELEASE', verbose=False):
1281+
product_type=None, dataset_release='LEVEL_3_RELEASE', verbose=False):
12831282
""" Gets the LE3 products (the high-level science data products).
12841283
12851284
Please note that not all combinations of category, group, and product_type are valid.
@@ -1547,7 +1546,7 @@ def get_scientific_data_product_list(self, *, observation_id=None, tile_index=No
15471546
#. 'DpdReferenceSample':
15481547
#. 'DpdTomographicBins':
15491548
1550-
release : str, mandatory.
1549+
dataset_release : str, mandatory. Default LEVEL_3_RELEASE
15511550
Data release from which data should be taken.
15521551
verbose : bool, optional, default 'False'
15531552
flag to display information about the process
@@ -1564,7 +1563,7 @@ def get_scientific_data_product_list(self, *, observation_id=None, tile_index=No
15641563
None):
15651564
raise ValueError("Include a valid parameter to retrieve a LE3 product.")
15661565

1567-
if release is None:
1566+
if dataset_release is None:
15681567
raise ValueError("The release is required.")
15691568

15701569
if observation_id is not None and tile_index is not None:
@@ -1671,7 +1670,7 @@ def get_scientific_data_product_list(self, *, observation_id=None, tile_index=No
16711670
f"observation_id_list, CAST(basic_download_data.tile_index_list as text) AS tile_index_list, "
16721671
f"CAST(basic_download_data.patch_id_list as text) AS patch_id_list, "
16731672
f"CAST(basic_download_data.filter_name as text) AS filter_name FROM sedm.basic_download_data WHERE "
1674-
f"release_name='{release}' {query_extra_condition} ORDER BY observation_id_list ASC")
1673+
f"release_name='{dataset_release}' {query_extra_condition} ORDER BY observation_id_list ASC")
16751674

16761675
job = super().launch_job(query=query, output_format='votable_plain', verbose=verbose,
16771676
format_with_results_compressed=('votable_gzip',))

astroquery/esa/euclid/tests/test_euclidtap.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1108,7 +1108,7 @@ def test_get_scientific_data_product_list_exceptions():
11081108
product_type=None)
11091109

11101110
with pytest.raises(ValueError, match="The release is required."):
1111-
eculid.get_scientific_data_product_list(observation_id=11111, release=None)
1111+
eculid.get_scientific_data_product_list(observation_id=11111, dataset_release=None)
11121112

11131113
with pytest.raises(ValueError, match="Incompatible: 'observation_id' and 'tile_id'. Use only one."):
11141114
eculid.get_scientific_data_product_list(observation_id=11111, tile_index=1234567)

0 commit comments

Comments
 (0)