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

Error in graphics::par(old.par) #4

Closed
cvitolo opened this issue Dec 18, 2016 · 5 comments
Closed

Error in graphics::par(old.par) #4

cvitolo opened this issue Dec 18, 2016 · 5 comments

Comments

@cvitolo
Copy link
Contributor

cvitolo commented Dec 18, 2016

This issue is part of this ropensci review

@ledell said:

Found a possible bug:
library(raster)
Loading required package: sp
b <- brick('trmm_acc.tif')
plot(b)
Error in graphics::par(old.par) :
invalid value specified for graphical parameter "pin"

@cvitolo
Copy link
Contributor Author

cvitolo commented Dec 18, 2016

@ledell I cannot reproduce your error here.

Have you generated the file trmm_acc.tiff using the TRMM() function before running the brick() command?

@mdsumner suggested to change slightly the workflow for the TRMM function.
Please install the latest version of hddtools from github and test the TRMM function using the example code below. Please let me know if the problem persists, thanks!

# Define a bounding box
areaBox <- raster::extent(-10, 5, 48, 62)
# Define a temporal extent
twindow <- seq(as.Date("2012-01-01"), as.Date("2012-03-31"), by = "months")
# Retreive mean monthly precipitations from 3B43_V7 (based on a bounding box and time extent)
TRMMfile <- TRMM(twindow = twindow, areaBox = areaBox)
library(raster)
plot(TRMMfile)

@mdsumner
Copy link

Seems fine now, here on Windows.

@cvitolo
Copy link
Contributor Author

cvitolo commented Dec 28, 2016

@mdsumner many thanks!

@ledell is this working for you too?

@ledell
Copy link

ledell commented Dec 30, 2016

@cvitolo Yes, this is working on my end (on a mac).

@cvitolo
Copy link
Contributor Author

cvitolo commented Jan 2, 2017

Great, thanks! I'm closing this now :)

@cvitolo cvitolo closed this as completed Jan 2, 2017
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

No branches or pull requests

3 participants