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

Fixed bug in get_ereefs_ts() in mmp check. #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

BarbaraRobson
Copy link
Collaborator

@ #34

@dbarneche
Copy link
Contributor

Hi Barbara, I just installed ncdf4 1.19 and ereefsfrom your fix, but it still seems to be failing.

> get_ereefs_ts('Chl_a_sum', location_latlon=data.frame(latitide=c(-23.39189,-18), longitude=c(150.88852, 147.5)), layer='surface', start_date=c(2010,12,31),end_date=c(2011,1,5), input_file=2)

  |                                                                                                                                  |   0%oc_open: server error retrieving url: code=403 message="Request too big=69895.440656 Mbytes, max=500.0"Error in Rsx_nc4_get_vara_double: NetCDF: Authorization failure
Var: Chl_a_sum  Ndims: 4   Start: 30,46,46,268 Count: 1,1,11,167
Error in ncvar_get_inner(ncid2use, varid2use, nc$var[[li]]$missval, addOffset,  : 
  C function R_nc4_get_vara_double returned error
In addition: Warning message:
In get_ereefs_ts("Chl_a_sum", location_latlon = data.frame(latitide = c(-23.39189,  :
  Assuming that the first column of location_latlon is latitude and the second column is longitude
...

And it keeps trying with the same error over and over.

Also, the function warn on this line does not exist, it needs to be warning

@dbarneche
Copy link
Contributor

The third example mentioned in #34 also failed

> get_ereefs_ts(var_names=c('ZooL_N', 'ZooS_N'), location_latlon=data.frame(latitide=c(-23.39189,-18), longitude=c(150.88852, 147.5)), layer=45, start_date=c(2010,12,31),end_date=c(2011,1,5), input_file="http://dapds00.nci.org.au/thredds/dodsC/fx3/gbr4_bgc_GBR4_H2p0_B2p0_Cpre_Dcrt/gbr4_bgc_simple_2016-06.nc")

  |                                                                                                                                  |   0%oc_open: server error retrieving url: code=403 message="Request too big=69895.440656 Mbytes, max=500.0"Error in Rsx_nc4_get_vara_double: NetCDF: Authorization failure
Var: ZooL_N  Ndims: 4   Start: 30,44,46,268 Count: 1,1,11,167
Error in ncvar_get_inner(ncid2use, varid2use, nc$var[[li]]$missval, addOffset,  : 
  C function R_nc4_get_vara_double returned error
In addition: Warning message:
In get_ereefs_ts(var_names = c("ZooL_N", "ZooS_N"), location_latlon = data.frame(latitide = c(-23.39189,  :
  Assuming that the first column of location_latlon is latitude and the second column is longitude
[1] "retrying in  1 second(s)"

@BarbaraRobson
Copy link
Collaborator Author

BarbaraRobson commented May 22, 2023 via email

@dbarneche
Copy link
Contributor

Trying with current ncdf 1.21 also fails. I'm not sure which one is the desired version?

@BarbaraRobson
Copy link
Collaborator Author

BarbaraRobson commented May 22, 2023 via email

@dbarneche
Copy link
Contributor

I just tried with devtools::install_github("mdsumner/ncdf4") but no luck.

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.

2 participants