Skip to content

Commit 678b20e

Browse files
committed
Implement review suggestions
1 parent 30516c3 commit 678b20e

3 files changed

Lines changed: 6 additions & 6 deletions

File tree

CHANGES.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ Service fixes and enhancements
3737
svo_fps
3838
^^^^^^^
3939

40-
- Add ``get_filter_params`` to allow retrieval of filter metadata. [#3528]
40+
- Add ``get_filter_metadata`` to allow retrieval of filter metadata. [#3528]
4141

4242
heasarc
4343
^^^^^^^

astroquery/svo_fps/core.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,8 @@ def get_filter_index(self, wavelength_eff_min, wavelength_eff_max, **kwargs):
108108
"succeed. Try increasing the timeout limit if a large range is needed."
109109
)
110110

111-
def get_filter_params(self, filter_id, cache=True, timeout=None):
112-
f"""Get metadata/parameters for the requested Filter ID from SVO
111+
def get_filter_metadata(self, filter_id, *, cache=True, timeout=None):
112+
"""Get metadata/parameters for the requested Filter ID from SVO
113113
114114
Parameters
115115
----------
@@ -121,7 +121,7 @@ def get_filter_params(self, filter_id, cache=True, timeout=None):
121121
Defaults to True. If set overrides global caching behavior.
122122
See :ref:`caching documentation <astroquery_cache>`.
123123
timeout : int
124-
Timeout in seconds. If not specified, defaults to {conf.timeout}.
124+
Timeout in seconds. If not specified, defaults to ``conf.timeout``.
125125
126126
Returns
127127
-------

astroquery/svo_fps/tests/test_svo_fps_remote.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ def test_get_transmission_data(self, test_filter_id):
2525

2626
@pytest.mark.parametrize('test_filter_id',
2727
['NewHorizons/MVIC.Blue', 'Palomar/ZTF.r'])
28-
def test_get_filter_params(self, test_filter_id):
29-
params = SvoFps.get_filter_params(test_filter_id)
28+
def test_get_filter_metadata(self, test_filter_id):
29+
params = SvoFps.get_filter_metadata(test_filter_id)
3030
# Check if expected keys are present
3131
assert "WavelengthEff" in params
3232
assert "ZeroPoint" in params

0 commit comments

Comments
 (0)