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

mergeRaster improvements #283

Closed
achubaty opened this Issue May 31, 2016 · 0 comments

Comments

Projects
None yet
1 participant
@achubaty
Contributor

achubaty commented May 31, 2016

mergeRaster (#282) should preserve as many raster attributes as possible:

  • layer name (names(r))
  • min and max (i.e., range of) values (min(values(r)) and max(values(r)) )

* note that the tests will need to be updated

achubaty added a commit that referenced this issue May 31, 2016

clean up PR #282
* add and improve tests for `splitRaster` and `mergeRaster`
* improve documentation
* improve code formatting

TO DO: see #283

@achubaty achubaty added the enhancement label Jun 1, 2016

@achubaty achubaty modified the milestone: v1.2.0 Jun 21, 2016

achubaty added a commit that referenced this issue Jun 22, 2016

fix & cleanup PR #286
* `mergeRaster` improvements (#283)
* `splitRaster` improvements (#284)
* moved shared examples to `inst/examples/examples_splitRaster.R`
* rename some arguments to make debugging easier
* don't write raster after doing all cropping; instead crop then write each raster tile in sequence
* cleanup

achubaty added a commit that referenced this issue Jun 22, 2016

fix & clean up split/merge raster tests
* move parallel splitRaster test into a separate test (skipped because needs to be run manually)
* remove 'TO DO' comments for issues #283 & #284 which are complete
* clean up code used in each test

@achubaty achubaty closed this Jun 22, 2016

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