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

enable chunking of images #42

Open
ircwaves opened this issue Jan 17, 2019 · 3 comments
Open

enable chunking of images #42

ircwaves opened this issue Jan 17, 2019 · 3 comments

Comments

@ircwaves
Copy link
Contributor

Description:

Given a temporally dense time series, over a wide enough area, multitemporal's memory requirements can move into the realm of tooo much. Especially in the case where you will output an entire time series (after working some magic on it) -- that sends things passed 20GiB.

Current implementation

reads in block-wise, but holds all the data until the end and writes out everything in one shot.

Proposed resolution:

write out data block-wise as well.

@bhbraswell @justinfisk -- seems reasonable, no?

@bhbraswell
Copy link

@bhbraswell @justinfisk -- seems reasonable, no?

That seems totally reasonable and good as far as can tell.

@ircwaves
Copy link
Contributor Author

ircwaves commented Feb 4, 2019

offline, @justinfisk mentioned that there might have been some reason that the output was not done in chunks. Is that ringing any bells for either of you?

@bhbraswell
Copy link

My recollection is that after some initial discussions, at which point the current framework already existed, I just didn't want to write the code that would save and then reassemble the chunks. So, basically laziness. As long as each step in the processing chain has the data it needs then I can't think of a reason not to keep everything as chunks until the end if that is more performant.

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

2 participants