Skip to content
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

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

jcaracappa1
Copy link
Contributor

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)

@jcaracappa1 jcaracappa1 requested a review from BBeltz1 December 9, 2024 22:21
@BBeltz1 BBeltz1 requested review from sgaichas and andybeet December 10, 2024 14:43
@andybeet
Copy link
Member

andybeet commented Dec 10, 2024

after sourcing the function then running the following (Note: save_clean = F)

a <- get_thermal_habitat_gridded(threshold = c(15,18.5,24),
                            year = 2023,
                            depth.bins = c(0,25,100,3000),
                            save_clean = F)

Then plotting this object using the plot function results in this

image

@sgaichas
Copy link
Member

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

@andybeet
Copy link
Member

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

@jcaracappa1
Copy link
Contributor Author

This looks fixed now
image

@andybeet
Copy link
Member

andybeet commented Dec 11, 2024

a <- get_thermal_habitat_gridded(threshold = c(15,18.5,24),
                            year = 2023,
                            depth.bins = c(0,25,100,3000),
                            save_clean = F)

image

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.

@jcaracappa1
Copy link
Contributor Author

I think it's broken because Kim is reuploading the data. It's breaking for me too. I'll test again soon

@jcaracappa1
Copy link
Contributor Author

This should be fixed now

@andybeet
Copy link
Member

still result in error for me and Brandon

@khyde
Copy link

khyde commented Dec 13, 2024

@andybeet When did you try it? The ERDDAP server is currently down because of the generator work.

@andybeet
Copy link
Member

just tried again. still errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants