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

splitRaster can not split the whole raster #276

Closed
LidongMo opened this Issue May 21, 2016 · 7 comments

Comments

Projects
None yet
2 participants
@LidongMo

LidongMo commented May 21, 2016

I have a series of big raster of the global climate data. Each raster is 4.6 GB, and when execute the splitRaster(), the function split only the third quadrant. By the way there is a warning

In [<-(*tmp*, cnt, value = p@polygons[[i]]@polygons[[j]]) : implicit list embedding of S4 objects is deprecated

what's wrong with this ?

@achubaty

This comment has been minimized.

Contributor

achubaty commented May 21, 2016

That warning is new in R 3.3.0

I'll take a look. Thanks for reporting.

On Saturday, 21 May 2016, LidongMo notifications@github.com wrote:

I have a series of big raster of the global climate data. Each raster is
4.6 GB, and when execute the splitRaster(), the function split only the
fourth quadrant. By the way there is a warning

In [<-(tmp, cnt, value = p@polygons[[i]]@polygons
https://github.com/Polygons[[j]]) : implicit list embedding of S4
objects is deprecated

what's wrong with this ?


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#276

Alex Chubaty

@LidongMo

This comment has been minimized.

LidongMo commented May 21, 2016

Hi Alex,
I am sorry I have make a little mistake in reporting. It’s the third quadrant not the fourth.
thanks,
Lidong

在 2016年5月21日,下午4:28,Alex Chubaty notifications@github.com 写道:

That warning is new in R 3.3.0

I'll take a look. Thanks for reporting.

On Saturday, 21 May 2016, LidongMo notifications@github.com wrote:

I have a series of big raster of the global climate data. Each raster is
4.6 GB, and when execute the splitRaster(), the function split only the
fourth quadrant. By the way there is a warning

In [<-(tmp, cnt, value = p@polygons[[i]]@polygons
https://github.com/Polygons[[j]]) : implicit list embedding of S4
objects is deprecated

what's wrong with this ?


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#276

Alex Chubaty

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub #276 (comment)

@achubaty

This comment has been minimized.

Contributor

achubaty commented May 27, 2016

@LidongMo sorry for the delay in testing this. I've been trying to put together a reproducible example using a large raster on disk, and came across some performance bottlenecks I want to address in the process. Also, @YongLuo007 is preparing a pull request that adds a 'buffer' argument so that the split raster tiles can be buffered by some number of pixels. Together, these improvements may address your issue, but I'd appreciate it if you can please provide a reproducible example of your problem.

PS the source of the warning is in the user of raster::rasterize() and only appears in R 3.3.0 (this is new, and the raster package maintainers haven't yet addressed it).

@achubaty

This comment has been minimized.

Contributor

achubaty commented May 27, 2016

@LidongMo I pushed a fix to splitRaster (860f7ee). can you please try it out to confirm this solves your issue?

@LidongMo

This comment has been minimized.

LidongMo commented May 27, 2016

Hi Alex,
I am sorry that I cannot execute a test yet. Because I am at home now(Europe ). I will try it the next week, and send you the error code I get before.
Thank you very much.

Have a nice weekend.
Lidong

发自我的 iPhone

在 2016年5月28日,上午12:18,Alex Chubaty notifications@github.com 写道:

@LidongMo I pushed a fix to splitRaster (860f7ee). can you please try it out to confirm this solves your issue?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@achubaty achubaty added the bug label Jun 1, 2016

@achubaty

This comment has been minimized.

Contributor

achubaty commented Jun 8, 2016

@LidongMo can you please provide an update on this? I'd like to close this issue if it's resolved. Thank you!

@achubaty

This comment has been minimized.

Contributor

achubaty commented Jun 13, 2016

closing, as I cannot reproduce this bug in the development branch.

@achubaty achubaty closed this Jun 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment