Fleshed out images vignette#86
Conversation
Chris-Schnaufer
requested review from
KristinaRiemer,
dlebauer and
kimberlyh66
dlebauer
left a comment
dlebauer
left a comment
There was a problem hiding this comment.
- need to figure out how to handle API keys to maximize user engagement
- suggested additional notes re: requirements / assumptions
|
|
||
| As mentioned in the overview, the url string will point to the API to use. In this case | ||
| we'll be using "https://terraref.ncsa.illinois.edu/clowder/api" and the key will be the | ||
| one you received in an email. |
There was a problem hiding this comment.
... will users have received this in the email? Do we need to tell them to sign up? @max-zilla do you have suggestions for how to access the data that we are going to make public?
There was a problem hiding this comment.
I think we need to tell them to sign up. I will update the doc
Co-Authored-By: Chris-Schnaufer <schnaufer@email.arizona.edu>
| sensor = 'RGB GeoTIFFs Datasets' | ||
| sitename = '' | ||
| files = get_file_listing(None, url, key, sensor, sitename, | ||
| since='2018-05-01', until='2018-05-31') |
There was a problem hiding this comment.
even after adding my API key, I get the following error:
Traceback (most recent call last):
File "", line 2, in
File "/usr/local/lib/python3.7/site-packages/terrautils/products.py", line 157, in get_file_listing
sens = get_sensor(connection, host, key, sensor, sitename)
File "/usr/local/lib/python3.7/site-packages/terrautils/products.py", line 101, in get_sensor
r.raise_for_status()
File "/usr/local/lib/python3.7/site-packages/requests/models.py", line 940, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://terraref.ncsa.illinois.edu/clowder/apiapi/geostreams/streams?key=MyAPIKEY&stream_name=RGB+GeoTIFFs+Datasets
There was a problem hiding this comment.
It looks like you have 'apiapi' in the URL. Is that a copy/paste error or is something not clear?
2 participants
Something more to review