[GRASS-dev] [GRASS GIS] #3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress

GRASS GRASS developer
1

#3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress
----------------------------------+---------------------------------
Reporter: sbl | Owner: grass-dev@…
     Type: defect | Status: new
Priority: normal | Milestone: 7.6.1
Component: Docs | Version: svn-releasebranch76
Keywords: r.compress,variables | CPU: All
Platform: All |
----------------------------------+---------------------------------
The release news for GRASS 7.6:
https://trac.osgeo.org/grass/wiki/Release/7.6.0-News , say that new
default raster compressor (if available) is ZSTD.

Manuals for:
* variables (https://grass.osgeo.org/grass76/manuals/variables.html)
* r.compress (https://grass.osgeo.org/grass76/manuals/r.compress.html)
* ???

are not fully consistent/up to date in this regards and should be updated
accordingly...

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3758&gt;
GRASS GIS <https://grass.osgeo.org>

2

#3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress
---------------------+----------------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.6.1
Component: Docs | Version: svn-releasebranch76
Resolution: | Keywords: r.compress,variables
       CPU: All | Platform: All
---------------------+----------------------------------

Comment (by mmetz):

In [changeset:"74114" 74114]:
{{{
#!CommitTicketReference repository="" revision="74114"
update documentation with regard to ZSTD as default compression if
available, see #3758
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3758#comment:1&gt;
GRASS GIS <https://grass.osgeo.org>

3

#3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress
---------------------+----------------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.6.1
Component: Docs | Version: svn-releasebranch76
Resolution: | Keywords: r.compress,variables
       CPU: All | Platform: All
---------------------+----------------------------------

Comment (by mmetz):

In [changeset:"74115" 74115]:
{{{
#!CommitTicketReference repository="" revision="74115"
update documentation with regard to ZSTD as default compression if
available, see #3758 (backport trunk r74114)
}}}

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3758#comment:2&gt;
GRASS GIS <https://grass.osgeo.org>

4

#3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress
---------------------+----------------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.6.1
Component: Docs | Version: svn-releasebranch76
Resolution: | Keywords: r.compress,variables
       CPU: All | Platform: All
---------------------+----------------------------------

Comment (by sbl):

Thanks. Found one little glitch (see attached diff for a suggestion).
BTW., the manual for r.compress
(https://trac.osgeo.org/grass/browser/grass/trunk/raster/r.compress/r.compress.html?rev=74114#L98)
says:

{{{
The NULL file compression is using the LZ4 method as being the best
compromise between speed and compression rate.
}}}

Is that still accurate?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3758#comment:3&gt;
GRASS GIS <https://grass.osgeo.org>

5

#3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress
---------------------+----------------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.6.1
Component: Docs | Version: svn-releasebranch76
Resolution: | Keywords: r.compress,variables
       CPU: All | Platform: All
---------------------+----------------------------------
Changes (by sbl):

* Attachment "r.compress.html.diff" added.

Removing last ZLIB-default remnant

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3758&gt;
GRASS GIS <https://grass.osgeo.org>

6

#3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress
---------------------+----------------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: defect | Status: new
  Priority: normal | Milestone: 7.6.1
Component: Docs | Version: svn-releasebranch76
Resolution: | Keywords: r.compress,variables
       CPU: All | Platform: All
---------------------+----------------------------------

Comment (by mmetz):

Replying to [comment:3 sbl]:
> Thanks. Found one little glitch (see attached diff for a suggestion).

Thanks, applied in trunk r74119. You can of course submit these
improvements yourself :wink:

> BTW., the manual for r.compress
(https://trac.osgeo.org/grass/browser/grass/trunk/raster/r.compress/r.compress.html?rev=74114#L98)
says:
>
> {{{
> The NULL file compression is using the LZ4 method as being the best
compromise between speed and compression rate.
> }}}
>
> Is that still accurate?

Yes, there is practically no speed penalty with LZ4, and NULL files are so
simple that not much is gained with other compression methods.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3758#comment:4&gt;
GRASS GIS <https://grass.osgeo.org>

7

#3758: zstd-suport in GRASS >= 7.6: update manuals for variables and r.compress
---------------------+----------------------------------
  Reporter: sbl | Owner: grass-dev@…
      Type: defect | Status: closed
  Priority: normal | Milestone: 7.6.1
Component: Docs | Version: svn-releasebranch76
Resolution: fixed | Keywords: r.compress,variables
       CPU: All | Platform: All
---------------------+----------------------------------
Changes (by sbl):

* status: new => closed
* resolution: => fixed

Comment:

Thanks for clarification.

Yes, next time I open tickets esp. on documentation I try to come up with
a fix myself much faster. (But you make it hard to be fast enough ;-))

Closing as fixed.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3758#comment:5&gt;
GRASS GIS <https://grass.osgeo.org>