[GRASS-dev] [GRASS GIS] #2750: LZ4 when writing raster rows; better than double I/O bound r.mapcalc speed

GRASS GIS trac at osgeo.org
Sun Oct 18 11:47:31 PDT 2015


#2750: LZ4 when writing raster rows; better than double I/O bound r.mapcalc speed
--------------------------+---------------------------
  Reporter:  sprice       |      Owner:  grass-dev@…
      Type:  enhancement  |     Status:  new
  Priority:  normal       |  Milestone:  7.1.0
 Component:  Raster       |    Version:  svn-trunk
Resolution:               |   Keywords:  ZLIB LZ4 ZSTD
       CPU:  OSX/Intel    |   Platform:  MacOSX
--------------------------+---------------------------

Comment (by wenzeslaus):

 Replying to [comment:17 sprice]:
 > My main concern really is to get this merged & working before additional
 changes such as those discussed in #2762 & #2349 make merging difficult.

 That's right, but I would like to see some more testing before merging it.
 Any chance somebody can do some? There are roughly three ways how to do
 that after applying the patch:

  * Set the variables (see above) and run the automatic tests on NC SPM and
 report the results back.
  * Run the benchmark Bash script attached to this ticket and compare with
 results already reported.
  * Set the variables and go through some your workflow and check the
 results.

 It seems to be that #2349 has all issues solved and committed, but lacks
 feedback/testing. Conflicts with #2762 would be probably solvable, but
 yes, it would be better to avoid them.

 > What do you think about merging these changes now, and then upgrading
 other I/O functions as you have time?

 This can work. The refactoring I'm suggesting shouldn't be difficult at
 the end, although we should take some time to do it right.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/2750#comment:18>
GRASS GIS <https://grass.osgeo.org>



More information about the grass-dev mailing list