Opened 6 years ago

Last modified 4 years ago

#3645 new defect

r.neighborhoodmatrix: inverse meaning of -c flag in grass8

Reported by: mlennert Owned by: grass-dev@…
Priority: normal Milestone: 7.8.3
Component: Addons Version: unspecified
Keywords: r.neighborhoodmatrix Cc:
CPU: Unspecified Platform: Unspecified

Description

This ticket is just there as a reminder that when grass8 comes out we should use the opportunity to inverse the meaning of the -c flag in r.neighborhoodmatrix to harmonize it with other modules.

-c should indicate: do not show column headers. Currently it indicates: show column headers.

Change History (5)

comment:1 by wenzeslaus, 6 years ago

That's fine with me, but you may also want to consider that:

  1. r.neighborhoodmatrix is an addon module, so there is no guarantee of stable API linked with releases of GRASS GIS, i.e. you can change it anytime (Addons provide option to rapidly change, but in public and that's what makes them different from core.)
  1. You can introduce two flags, one to show and one to hide the headers. For 7, one will be NOP and the other will show column header, for 8 it will be the reverse. So that people who care about heaving or not heaving the header can actually indicate it by a flag and their code will give the right result with both 7 and 8 versions of the module.

comment:2 by mlennert, 5 years ago

Summary: r.nieghborhoodmatrix: inverse meaning of -c flag in grass8r.neighborhoodmatrix: inverse meaning of -c flag in grass8

in reply to:  1 comment:3 by mlennert, 5 years ago

I don't know why, but I completely missed that you had responded, Vaclav.

Replying to wenzeslaus:

That's fine with me, but you may also want to consider that:

  1. r.neighborhoodmatrix is an addon module, so there is no guarantee of stable API linked with releases of GRASS GIS, i.e. you can change it anytime (Addons provide option to rapidly change, but in public and that's what makes them different from core.)

Even though I agree in principle, in practice I know that some people use addons also in scripts and changing the API can have consequences on these scripts, so I think we should be careful about this.

  1. You can introduce two flags, one to show and one to hide the headers. For 7, one will be NOP and the other will show column header, for 8 it will be the reverse. So that people who care about heaving or not heaving the header can actually indicate it by a flag and their code will give the right result with both 7 and 8 versions of the module.

Good point. I'll think about this option.

comment:4 by martinl, 5 years ago

Milestone: 8.0.0

Remove Milestone from Addons bugreports.

comment:5 by neteler, 4 years ago

Milestone: 7.8.3
Note: See TracTickets for help on using tickets.