-
Notifications
You must be signed in to change notification settings - Fork 12
update date format #256
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update date format #256
Conversation
|
FYI @LindsayRAbrams. Curious if you know who manages https://opendap.co-ops.nos.noaa.gov/erddap/index.html? |
|
The CO-OPS ERDDAP is very non-standard in that it requires the extra STATION_ID and possibly other parameters to return results. It's been awhile since I've looked into it, but I recall it's a design limitation within the CO-OPS database that requires such parameters by design in order to return data. Maybe they have an alternate data source at this point that could replace this that ERDDAP could run from. Although looking at the error message you included @ocefpaf, it looks like it's PostgreSQL now, whereas last I looked it was still running Sybase, so maybe they've updated but the underlying parameter requirement is unchanged. I got the 504 timeout error when trying to access the ERDDAP homepage link above just now. So it may be that it's just not operationally supported at the moment too. |
Indeed. Not only that, they must be double quoted!
I'm not sure if that is a red herring or a real error. Updating the date format fixes it, but there is still the matter of variable order in the URL that should not play a role in the query. (I may be wrong here. I haven't seem any ERDDAP server that requires variables to be in a fixed order.) See https://gist.github.com/ocefpaf/bb0fd4551d5c8ce91f4b778c8cc54ec3 for an example. |
84f659a to
5f96432
Compare
|
Needs ioos/erddapy#399 |
|
The |
This server is still on ERDDAP 2.18 and has a non-standard date/time variable. The format change, but we are also experiencing erros when the variable and constraints are in a different order:
Those two should be the same in ERDDAP, but here one works and the other fails with: