Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
nabobalis authored Sep 9, 2023
1 parent fea12d5 commit 32b4f76
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
5 changes: 3 additions & 2 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,9 @@ Installation
------------

**Note**

With the updates to the SOAR, the product names now need to be lowercase.
This is fixed with the latest release of sunpy-soar, so you may need to update if
This is fixed with the latest release of sunpy-soar, so you will need to update if
you are having issues finding data.

sunpy-soar requires `python >= 3.7` and `sunpy >= 2.1`.
Expand Down Expand Up @@ -53,7 +54,7 @@ The code below gives an example of how to search and download Solar Orbiter data
instrument = a.Instrument('EUI')
time = a.Time('2021-02-01', '2021-02-02')
level = a.Level(1)
product = a.soar.Product('eui-fsi174-image')
product = a.soar.Product('EUI-FSI174-IMAGE')
# Do search
result = Fido.search(instrument & time & level & product)
Expand Down
1 change: 1 addition & 0 deletions sunpy_soar/attrs.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
class Product(SimpleAttr):
"""
The data product identifier to search for.
Makes the value passed lower so that it is case insensitive as
all descriptors on the SOAR are now lowercase.
"""
Expand Down

0 comments on commit 32b4f76

Please sign in to comment.