Bugfix providers even further#8
Conversation
|
Thanks for all the fixes! There was a reason the Do you want to be a maintainer here? This package was hacked together very quickly while we put together Tyler.jl. It could do with some more ongoing care, but I'm unlikely to find time for it. |
| end | ||
|
|
||
|
|
||
| PROVIDER_DICT[NASAGIBSTimeseries] = _NASAGIBS_KEYS No newline at end of file |
There was a problem hiding this comment.
(optional) Can you include a comment here to explain this line? Also, github's review UI complains when the file does not end with a newline
There was a problem hiding this comment.
Yes. I will do that. This is just adding the nasagibstimeseries to the main dict of providers (as you probably know...). I will also add a new line.
|
@SuperGrobi might need to get out an old 1.6 version and see whats causing that failure. Probably one of the line you have added like the nada gibbs tests? Something is wrong with |
|
Sorry I just realised this was against my other PR that I totally forgot existed, and not master |
I fixed a bunch of problems and inconsistencies I came across when playing around with different tile providers. Here they are:
In the NASAGIBS file:
NASAGIBSTimeseriesfailed due to the default keyword not being in the list of valid ones.NASAGIBSTimeseries(still, somehow printing about 800 lines of variantlist makes the docstring basically useless in the REPL...)NASAGIBSTimeseriestoPROVIDER_DICTin the providers file:
keywordin the docstring to always be lowercase, since the keywords of the functions are as wellvariantone line up, otherwise we get thekeywordof theProviderto show up as a possiblevariantin the error messagemethodsfunction (seemethods(f)not showing correct signatures for functions with default arguments JuliaLang/julia#49028)Tests:
Providersin the order they appear in thejsonReadme:
PROVIDER_DICT