-
Notifications
You must be signed in to change notification settings - Fork 0
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
Include the BCCM output for its whole domain #49
Comments
Similar request from Eva and Brianna: Brianna and I are hoping to use the BCCM data from the pacea package for some spatial modelling but our model extent goes beyond the Canadian EEZ (specifically we need to include WA state waters). Is it possible to remove the EEZ mask when calling pacea::bccm_surface_temperature() to get the full geographic extent of Pena et al.’s (2019) model predictions? |
@LisaCrozier-NOAA - I am working on this now. So for the BC waters we made a 2x2 km inshore grid and a 6x6 offshore grid, to save storage space. I just want to check if a similar thing is sufficient for your needs, with the grid extended down to the US? Basically, covering the full domain of Angelica’s model. |
Here's the current 2x2 inshore grid (green) and 6x6 offshore grid (blue) currently used for restricting the BCCM outputs to BC waters. So the idea would be something similar extending south (to save storage space, I think 2x2 everywhere will get unwieldly). And see the BCCM vignette for what functionality we have built in already: https://htmlpreview.github.io/?https://github.com/pbs-assess/pacea/blob/main/vignettes/bccm.html My aim is to make all this work fine for the full domain results. |
Okay, I may just stick with 2x2 throughout as it's a bit simpler to deal with. Filesizes will be large (but only need to be downloaded once), so I've been investigating ways of hosting them on Zenodo rather than GitHub and adapting some of our existing R functions to download them. Have successfully saved the .rds file for Then all the functionality in Travis's BCCM vignette will work, such as:
|
Have locally built all the BCCM files (for different variables) at 2x2 km resolution over the full original domain. Need to upload then download them, but here's an example of what will be easy to look at (primary production for each month in 2018):
|
Done in the above commit. See the new vignette. |
To save storage space, we clipped the BCCM output to only cover the Canadian Exclusive Economic Zone.
Request from Lisa Crozier (@lisa439), arising from our DFO/NOAA meeting, to include the US waters, which should reach down to the Columbia River.
Given Travis figured out how to store the EEZ model outputs, I think the best way to do this would be to just save the full BCCM output in another data repository, such as pacea-data-full-bccm . Keep the structure of everything that's saved the same, and just have the variable names appended by
_full
so that we can have both sets of results stored locally on our computers. Then, the existing functions and vignettes should still work with variables appended by_full
, depending on how automatically maps etc. get resized.We may want to create a new GitHub organisation, given the email we got about Large File Size and being at 80% of capacity. This wouldn't matter as users don't actually need to interact with the data repository page, since Travis's functions call it directly when first downloading the BCCM results.
This will likely end up being more complex than outlined above, but just getting some ideas here for now.
The text was updated successfully, but these errors were encountered: