#2621: r.random.cell should use NULL instead of 0
-------------------------+--------------------------------------------------
Reporter: marisn | Owner: grass-dev@…
Type: defect | Status: new
Priority: minor | Milestone: 7.0.1
Component: Raster | Version: 7.0.0
Keywords: | Platform: Unspecified
Cpu: Unspecified |
-------------------------+--------------------------------------------------
It seems that r.random.cell is using 0 to mark empty areas instead of
using proper NULL values.
It is easy fixable by r.nulls call, still it should be fixed.
If it is a desired behaviour, documentation should be updated to state it.
It seems that r.random.cell is using 0 to mark empty areas instead of
using proper NULL values.
It is easy fixable by r.nulls call, still it should be fixed.
If it is a desired behaviour, documentation should be updated to state
it.
New description:
It seems that r.random.cells is using 0 to mark empty areas instead of
using proper NULL values.
It is easy fixable by r.null call, still it should be fixed.
If it is a desired behaviour, documentation should be updated to state it.
#2621: r.random.cells should use NULL instead of 0
----------------------------+-----------------------------------------------
Reporter: marisn | Owner: grass-dev@…
Type: defect | Status: new
Priority: normal | Milestone: 7.0.1
Component: Raster | Version: svn-releasebranch70
Keywords: r.random.cells | Platform: Unspecified
Cpu: Unspecified |
----------------------------+-----------------------------------------------
Comment(by marisn):
Replying to [comment:2 neteler]:
> * Backport to relbranch7
I would say - no. It changes behaviour of module. 7.0.x shouldn't change
such things as it might break some scripts relaying on consistency of 7.0.
#2621: r.random.cells should use NULL instead of 0
----------------------------+-----------------------------------------------
Reporter: marisn | Owner: grass-dev@…
Type: defect | Status: new
Priority: normal | Milestone: 7.0.1
Component: Raster | Version: svn-releasebranch70
Keywords: r.random.cells | Platform: Unspecified
Cpu: Unspecified |
----------------------------+-----------------------------------------------
Comment(by neteler):
G7.1 manual updated in r65024.
Replying to [comment:3 marisn]:
> Replying to [comment:2 neteler]:
> > * Backport to relbranch7
> I would say - no. It changes behaviour of module. 7.0.x shouldn't change
such things as it might break some scripts relaying on consistency of 7.0.
Maybe. However, the 7.0 manual states "The cells are numbered from 1 to
the
numbers of cells generated." which is not true.
#2621: r.random.cells should use NULL instead of 0
----------------------------+-----------------------------------------------
Reporter: marisn | Owner: grass-dev@…
Type: defect | Status: new
Priority: normal | Milestone: 7.0.1
Component: Raster | Version: svn-releasebranch70
Keywords: r.random.cells | Platform: Unspecified
Cpu: Unspecified |
----------------------------+-----------------------------------------------
Comment(by marisn):
Replying to [comment:4 neteler]:
> Maybe. However, the 7.0 manual states "The cells are numbered from 1 to
the
> numbers of cells generated." which is not true.
>
Then the manual needs to be fixed. (It also doesn't state that empty cells
have NULLs thus manual is not contradicting current behaviour.)
How about adding a simple sentence "Empty cells are marked with 0 (will be
changed to NULL in GRASS GIS 7.1.0 release)."?
I would say - fix documentation, add to GRASS 7.1 changes and close this
issue.