-
Notifications
You must be signed in to change notification settings - Fork 12
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
changed get_thermal_habitat_gridded to pull from ERDDAP before format… #179
base: dev
Are you sure you want to change the base?
Conversation
so looks like they are all the same plot, and the Value units aren't in days? catalog for reference: https://noaa-edab.github.io/catalog/thermal_habitat_persistence.html#key-results-and-visualizations-46 |
it does look like (from inspecting the data) that the data for each plot is identical. And yes very different from catalog. Must be a bug somewhere |
I think this is why the pulling and processing of the data should be in an external package (to ensure all dependencies are installed and to document if there are additional system requirements, looks like i might need a newer version of GDAL?). Then the package::function is used in the get function and all is needed is then to format the data. |
I think it's broken because Kim is reuploading the data. It's breaking for me too. I'll test again soon |
This should be fixed now |
still result in error for me and Brandon |
@andybeet When did you try it? The ERDDAP server is currently down because of the generator work. |
just tried again. still errors |
Added spatial data for GLORYS EPU mapping and bathymetry
Modified data-raw/get_thermal_habitat_gridded.R to pull from ERDDAP (need to be behind firewall to run)