Re: [GRASS-dev] [GRASS GIS] #2503: wxdigit: wrong undo redo

2014-12-03 Thread GRASS GIS
#2503: wxdigit: wrong undo  redo
-+--
 Reporter:  mlennert |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  wxGUI| Version:  svn-trunk
 Keywords:  digitizer|Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by mlennert):

 Replying to [comment:5 mmetz]:
  Replying to [comment:4 mlennert]:
   Replying to [comment:3 mmetz]:
Replying to [comment:2 mlennert]:

 There still is some confusion when one goes back one or two steps,
 then digitizes something else.
   
The question is what should happen with the available redo steps if
 digitize something new after some undo steps. Eliminate the redo steps or
 insert the new changeset before the first redo step? This is handled by
 the wx digitizer, not the vector lib.
  
   I've tried three different programs (LibreOffice Writer, Inkscape and
 GIMP) and all discard these redo steps,i.e. when you do A, then B, then
 undo B, then do C, you cannot find B again.
  
   This seems to be the most intuitive behaviour to me.
 
  OK.
  
   
 When you then undo and redo again, the stack seems to be mixed
 between the different paths, sometimes leading to objects left on screen
 even when going all the way back to the last possible undo.
   
It seems that a new changeset is appended after the last redo step
 if available, leading to a mix-up.
   
  
   Yes.
 
  Fixed in trunk r63341, please test.

 Much better !

 There is still one issue I've come upon: when features are automatically
 modified from the form they are digitized in because of overlaps, the
 second original feature causing the overlap seems to be erased from the
 undo stack, meaning that it remains, even if you undo all the way to
 origin or if you close without saving.

 To reproduce, digitize a line and then a second line that crosses that
 line. The lines are correctly split at the intersection. If you then undo
 the second line remains in its original (unsplit) form. The same is true
 for two overlapping polygons or a line crossing a polygon.

-- 
Ticket URL: https://trac.osgeo.org/grass/ticket/2503#comment:6
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by huhabla):

 I full agree with Vaclav.

 However, the modification of GRASS was already performed, regardless the
 fact that we still discuss the renaming of rast, rast3d and vect options.

 This, as i already stated before, completely broke the temporal framework.
 It is not only a matter of renaming option names, the whole internal
 temporal data type system and handling of different map types was broken
 because of this. What to do now with temporal module names? Must t.rast.*
 modules be renamed into t.raster.*? Or t.rast3d.* modules into
 t.3draster.*?

 Someone tried to fix the framework by simply renaming rast3d into
 3draster, but this did not solved the problem. The framework was still
 broken. Why was the testsuite not used to check these changes? Why do we
 have the test-framework anyway?
 Just a tiny hint: renaming the option n from v.random into npoints
 broke plenty of temporal vector module tests, since they are using PyGRASS
 to call v.random and PyGRASS can't handle abbreviations.

 I needed 5 hours to fix the temporal framework and i am pretty sure that i
 did not catch all issues that were introduced with the renaming.

 Putting GRASS in such broken a state without using the testuite to write
 and use tests or at least to verify the changes with existing tests is
 IMHO irresponsible. Especially if we plan to make a beta release.

 I am sorry for my harsh words, but i had to invest 5h for no good reason
 to keep on with my regular work.


 Replying to [comment:80 wenzeslaus]:
  Replying to [comment:79 glynn]:
   Replying to [comment:77 annakrat]:
Replying to [comment:76 cmbarton]:
 Could it cause a problem somewhere down the line to have this term
 beginning with a number--e.g. If it is used to name a temp file or
 something?
   
As I already said, it causes problems for Python because keyword
 parameter can't start with number. This is solvable by adding underscore
 and some special handling of this case, which is partially there already.
 It violates pep8.
  
   PEP8 is a style guide. There is no inherent reason why an argument
 name cannot start with an underscore. And we're not even talking about
 explicit arguments; such arguments will only ever be obtained via the
 **kwargs mechanism.
 
  The problem may come once you want to use parameter as an variable or
 member variable. In later case underscore would means private which is
 technique not limited to Python. I'm also afraid that this can hit us or
 somebody else in some other language or system. Almost nothing allows
 numbers at the beginning of identifiers. I also think that for 3D raster
 it is much more probable that you hit this issue. For example, how should
 I name variable in my script which holds 3D raster map name or its
 maximum? `_3draster_name`? `_3draster_max`? I can of course name my
 variables whatever I want but wouldn't we stick to `rast3d` or `raster3d`
 in the GRASS source code anyway?
 
  
   In fact, I think that this is why I chose to use a leading underscore
 rather than a trailing underscore.
  
   Still, I'd rather avoid having option names start with a digit. But
 unless we relax the ambiguity check, it wouldn't outweigh my preference to
 avoid using an option name which has a very common option name (rast or
 raster) as a prefix.
 
  I'm glad you are saying that. I think it is really important to state
 the priorities and motivations. If we just want backwards compatibility,
 then some special check in the parser can handle old short option names.
 And if we value the most backwards compatibility and short options, we
 probably should not not shorten at all in these special cases (type
 names).
 
  Perhaps it is useful to ask why we want short options. It is for manual
 typing? Well then we perhaps should use techniques used elsewhere. And we
 are actually partially doing it. There is IDE-like auto-complete in GUI,
 Python dir completed is implemented for PyGRASS module interface and of
 course Linux command line auto-completes module names. So why not to take
 it further and auto-complete also parameters and perhaps other things by
 implementing auto-complete for shell?
 
  Classic unix-like command line is anyway the only place where short
 options really matter if you consider the things above and also that you
 should not use shortened option names in scripts because 

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by martinl):

 Replying to [comment:85 huhabla]:
  I needed 5 hours to fix the temporal framework and i am pretty sure that
 i did not catch all issues that were introduced with the renaming.

 sorry for that, *BUT* we were speaking about consolidation of parameters
 *MANY YEARS* ago! But *NOTHING* really *CHANGED* till now. I agree with
 you that we are too late with that but I still think that it made sense to
 do it.

  Putting GRASS in such broken a state without using the testuite to write
 and use tests or at least to verify the changes with existing tests is
 IMHO irresponsible. Especially if we plan to make a beta release.

 See my note above...

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:86
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by huhabla):

 Replying to [comment:86 martinl]:
  Replying to [comment:85 huhabla]:
   I needed 5 hours to fix the temporal framework and i am pretty sure
 that i did not catch all issues that were introduced with the renaming.
 
  sorry for that, *BUT* we were speaking about consolidation of parameters
 *MANY YEARS* ago! But *NOTHING* really *CHANGED* till now. I agree with
 you that we are too late with that but I still think that it made sense to
 do it.
 
   Putting GRASS in such broken a state without using the testuite to
 write and use tests or at least to verify the changes with existing tests
 is IMHO irresponsible. Especially if we plan to make a beta release.
 
  See my note above...

 Dear Martin,
 many thanks for all of your effort.

 I am not arguing about the sense of the changes. As i stated before i am
 fine with them. I simply have a problem how these changes are applied. We
 have a wonderful test suite that will help you to avoid breaking parts of
 GRASS. We should make use of it to avoid such a discussion.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:87
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2503: wxdigit: wrong undo redo

2014-12-03 Thread GRASS GIS
#2503: wxdigit: wrong undo  redo
-+--
 Reporter:  mlennert |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  wxGUI| Version:  svn-trunk
 Keywords:  digitizer|Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by mmetz):

 Replying to [comment:6 mlennert]:
 
  There is still one issue I've come upon: when features are automatically
 modified from the form they are digitized in because of overlaps, the
 second original feature causing the overlap seems to be erased from the
 undo stack, meaning that it remains, even if you undo all the way to
 origin or if you close without saving.
 
  To reproduce, digitize a line and then a second line that crosses that
 line. The lines are correctly split at the intersection. If you then undo
 the second line remains in its original (unsplit) form. The same is true
 for two overlapping polygons or a line crossing a polygon.

 The list of updated lines as returned by the vector lib was incomplete.
 Fixed in r63349,50 (trunk, relbr70), please test.

-- 
Ticket URL: https://trac.osgeo.org/grass/ticket/2503#comment:7
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

[GRASS-dev] [GRASS GIS] #2506: g.rename fails to rename map when there is already map of the same name in PERMENENT

2014-12-03 Thread GRASS GIS
#2506: g.rename fails to rename map when there is already map of the same name 
in
PERMENENT
-+--
 Reporter:  annakrat |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  Default  | Version:  svn-trunk
 Keywords:  g.rename |Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--
 In a mapset (different than PERMANENT) I run:

 {{{
 g.rename raster=mymap,elevation
 WARNING: elevation already exists in mapset PERMANENT
 }}}
 It doesn't rename it. When I use overwrite, it works. Still it seems it's
 a bug.

 Also when I add there my current mapset, it fails, and I thought we
 decided that current mapset should be supported when specifying output map
 names.

 {{{
 g.rename raster=mymap,elevation@mycurrentmapset
 WARNING: Illegal filename elevation@mycurrentmapset. Character @ not
 allowed.
 WARNING: elevation@mycurrentmapset is an illegal file name
 }}}

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2506
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

[GRASS-dev] Individual vector feature processing: v.clump?

2014-12-03 Thread Markus Neteler
Hi,

for a calculation I need to loop over all vector features in a map in
order to perform individually a computation. For that I cannot use the
category information since it may be used as 1:n link.

I wonder if a new v.clump command would make sense or the addition of
a new method in v.category to simplify this task?
Eventually I would implement the processing loop as a Python script.

Markus
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] [GRASS GIS] #2506: g.rename fails to rename map when there is already map of the same name in PERMENENT

2014-12-03 Thread GRASS GIS
#2506: g.rename fails to rename map when there is already map of the same name 
in
PERMENENT
-+--
 Reporter:  annakrat |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  Default  | Version:  svn-trunk
 Keywords:  g.rename |Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by martinl):

 The map is not renamed because G70 still uses abbreviated element names
 (ie. `rast`). `g.rename` silently fails because of
 source:grass/branches/releasebranch_7_0/lib/manage/list.c#L47

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2506#comment:1
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2506: g.rename fails to rename map when there is already map of the same name in PERMENENT

2014-12-03 Thread GRASS GIS
#2506: g.rename fails to rename map when there is already map of the same name 
in
PERMENENT
-+--
 Reporter:  annakrat |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  Default  | Version:  svn-trunk
 Keywords:  g.rename |Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by martinl):

 In GRASS 71 (trunk) it works.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2506#comment:2
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by martinl):

 Replying to [comment:87 huhabla]:
  many thanks for all of your effort.

 personally I am not happy with these changes too. They should have been
 done much more earlier, now it's very late. But still I think it's better
 than not to do it at all. Anyway I am not happy about this situation.

  I am not arguing about the sense of the changes. As i stated before i am
 fine with them. I simply have a problem how these changes are applied. We
 have a wonderful test suite that will help you to avoid breaking parts of
 GRASS. We should make use of it to avoid such a discussion.

 Let's try to focus on element list name. Under condition that we don't
 want to use abbreviated names, what is your preference

 {{{
 raster3d*
 volume
 3draster
 }}}

 (*) Then we could modify parser to understand also `rast` and `rast3d`
 with appropriate warning

 {{{
 g.list rast
 WARNING: Element rast has been renamed raster. Please update your
 script.
 ...

 g.list rast3d
 WARNING: Element rast3d has been renamed raster3d. Please update your
 script.
 ...
 }}}

 What do you think?

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:88
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS-SVN] r63227 - in grass/trunk/scripts: d.correlate d.out.file d.rast.leg d.redraw d.shadedmap d.to.rast d.vect.thematic d.what.rast d.what.vect

2014-12-03 Thread Glynn Clements

Vaclav Petras wrote:

   The issue is this:
  
   GRASS 7.1.svn (nc_spm_08_grass7):~  g.region rast=lsat7_2002_30
   GRASS 7.1.svn (nc_spm_08_grass7):~  d.correlate
  map=lsat7_2002_30,lsat7_2002_40
   ERROR: No graphics device selected. Use d.mon to select graphics device.
   Traceback (most recent call last):
   ...
   grass.exceptions.CalledModuleError: Module run None ['d.text',
   'color=black', 'line=1', 'stdin=CORRELATION', 'size=4'] ended with
   error
   Process ended with non-zero return code 1. See errors in the (error)
  output.
 
  I can't reproduce the problem. And I'm not even sure how d.text can
  exit with a non-zero return code without printing any error message.
 
 
 I can reproduce it. Are you sure you don't have MONITOR or
 GRASS_RENDER_IMMEDIATE (or any other if it matters) set? I start GRASS
 session (without any special shell environment) and I get it.
 
 The error is the beginning, r.text is printed first (that's ERROR:... I
 think), then the error from the main process is printed (Traceback).

Er, right; I got confused by the fact that the script changes used
exactly the same error message as d.text (which is a leftover from
6.x).

The issue has been fixed in r63361. The issue was first introduced in
r46984 and incorrectly fixed in r46999.

In GRASS 7, it isn't (or shouldn't have been) necessary to set
GRASS_RENDER_IMMEDIATE. This variable only exists to choose a driver
other than the default (cairo if supported, otherwise PNG).

 I think the solution is try-except or calling the function(s) with with
 secret `errors` parameter set to 'exit'.

No. There's no point in catching exceptions if you aren't going to
handle them. If we want to avoid the tracebacks, we should find a
solution which works globally, not require each and every script to
manually handle exceptions from each and every called module.

-- 
Glynn Clements gl...@gclements.plus.com
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] [GRASS GIS] #2503: wxdigit: wrong undo redo

2014-12-03 Thread GRASS GIS
#2503: wxdigit: wrong undo  redo
-+--
 Reporter:  mlennert |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  wxGUI| Version:  svn-trunk
 Keywords:  digitizer|Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by mlennert):

 Replying to [comment:7 mmetz]:
  Replying to [comment:6 mlennert]:
  
   There is still one issue I've come upon: when features are
 automatically modified from the form they are digitized in because of
 overlaps, the second original feature causing the overlap seems to be
 erased from the undo stack, meaning that it remains, even if you undo all
 the way to origin or if you close without saving.
  
   To reproduce, digitize a line and then a second line that crosses that
 line. The lines are correctly split at the intersection. If you then undo
 the second line remains in its original (unsplit) form. The same is true
 for two overlapping polygons or a line crossing a polygon.
 
  The list of updated lines as returned by the vector lib was incomplete.
 Fixed in r63349,50 (trunk, relbr70), please test.

 Still not quite:
 [http://tomahawk.ulb.ac.be/moritz/wxdigit_undo_redo4.ogv].

 If you need more explanation, I can write some, but right now I have to
 go.

 Moritz

-- 
Ticket URL: https://trac.osgeo.org/grass/ticket/2503#comment:8
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] addons dependencies

2014-12-03 Thread Moritz Lennert

On 01/12/14 18:07, Markus Neteler wrote:

On Mon, Dec 1, 2014 at 4:18 PM, Moritz Lennert
mlenn...@club.worldonline.be wrote:

On 28/11/14 23:41, Martin Landa wrote:

...

please try out r63275. Martin



v.in.wfs2 now runs (although it bails out because of the recent change in
parameters (v.in.ogr dsn - input).


I thought that this would be caught by the updated parser magic via
lib/gis/renamed_options?
Ah, v.in.ogr wasn't registered yet, so done in r63317 (and r63318 respectively).

Please try again,


Yes, now it works.

Only weird thing:

v.in.wfs2 url= http://geoserver.gis.irisnet.be/ows layers=urbis:URB_A_MZ 
output=urbis srs=31370

Downloading data from WFS server...
Reprojecting data...
Importing vector map into GRASS...
WARNING: Please update the interface of the module: option dsn has 
been renamed to input



For the user it is not clear where this warning comes from as dsn in not 
a parameter in v.in.wfs2. But this is true for all warnings of v.in.ogr 
in this module, so it is a different problem.


Moritz
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] addons dependencies

2014-12-03 Thread Martin Landa
Hi,

2014-12-03 19:07 GMT+01:00 Moritz Lennert mlenn...@club.worldonline.be:
 v.in.wfs2 url= http://geoserver.gis.irisnet.be/ows layers=urbis:URB_A_MZ
 output=urbis srs=31370
 Downloading data from WFS server...
 Reprojecting data...
 Importing vector map into GRASS...
 WARNING: Please update the interface of the module: option dsn has been
 renamed to input

please try out r63362.

WARNING: Please update the user interface of v.in.ogr: option dsn
has been renamed to input

Is it better? Martin

-- 
Martin Landa
http://geo.fsv.cvut.cz/gwiki/Landa
http://gismentors.eu/mentors/landa
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by glynn):

 Replying to [comment:84 wenzeslaus]:

  As I said before, short options are mostly advantageous for unix-like
 command lines, so let's use what it used there, the Tab key auto-complete.

 We shouldn't rely upon this feature being available. And even when it is,
 it's not automatically better than abbreviation (e.g. completion is
 affected by latency, abbreviation isn't).

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:89
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by glynn):

 Replying to [comment:88 martinl]:

  Let's try to focus on element list name. Under condition that we don't
 want to use abbreviated names, what is your preference
 
 {{{
 raster3d*
 volume
 3draster
 }}}

 My preference would be volume; the other two have issues. We can debate
 forever about how significant those issues are, but volume avoids them
 entirely.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:90
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2506: g.rename fails to rename map when there is already map of the same name in PERMENENT

2014-12-03 Thread GRASS GIS
#2506: g.rename fails to rename map when there is already map of the same name 
in
PERMENENT
-+--
 Reporter:  annakrat |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  Default  | Version:  svn-trunk
 Keywords:  g.rename |Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by wenzeslaus):

 Replying to [comment:2 martinl]:
  In GRASS 71 (trunk) it works.

 Not for me with older trunk. I have r62726 (before any renaming) and I'm
 getting:

 {{{
   r.mapcalc aaa = 1
  100%
   g.list rast mapset=.
 aaa
   g.rename rast=aaa,elevation
 WARNING: elevation already exists in mapset PERMANENT
   g.list rast mapset=.
 aaa
   g.rename rast=aaa,elevation --o
 Rename raster aaa to elevation
   g.list rast mapset=.
 elevation
 }}}


 {{{
 GRASS version: 7.1.svn
 GRASS SVN Revision: 62726M
 Build Date: 2014-11-18
 Build Platform: x86_64-unknown-linux-gnu
 GDAL/OGR: 1.10.1
 PROJ.4: 4.8.0
 GEOS: 3.4.2
 SQLite: 3.8.2
 Python: 2.7.6
 wxPython: 2.8.12.1
 Platform: Linux-3.13.0-40-generic-x86_64-with-Ubuntu-14.04-trusty
 }}}

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2506#comment:3
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by wenzeslaus):

 Replying to [comment:90 glynn]:
  Replying to [comment:88 martinl]:
 
   Let's try to focus on element list name. Under condition that we don't
 want to use abbreviated names, what is your preference
  
  {{{
  raster3d*
  volume
  3draster
  }}}
 
  My preference would be volume; the other two have issues. We can
 debate forever about how significant those issues are, but volume avoids
 them entirely.

 `volume` avoids issues with naming variables but creates issues when
 talking with people. From my experience, volume is very confusing while 3D
 raster (- `raster3d`) works because it is very explanatory and it is not
 a [http://en.wikipedia.org/wiki/Homonym homonym]. Voxel seems not be an
 option anymore but it is a term which people react to time to time because
 it is at least clear that it is something special (not a
 [http://en.wikipedia.org/wiki/Volume volume] in liters).

 BTW, Wikipedia is not really helpful in deciding about
 `volume`/`voxel`/`raster3d` but it links not really related
 [http://skuld.bmsc.washington.edu/raster3d/raster3d.html Raster3D]
 software ([http://en.wikipedia.org/wiki/Raster_graphics#External_links
 Raster graphics]) and it talks about ''2D and 3D raster and vector'' when
 it is describing a book
 
[http://en.wikipedia.org/wiki/Computer_Graphics:_Principles_and_Practice#Second_Edition
 Computer Graphics: Principles and Practice] which is highly related.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:91
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by annakrat):

 Replying to [comment:90 glynn]:
  Replying to [comment:88 martinl]:
 
   Let's try to focus on element list name. Under condition that we don't
 want to use abbreviated names, what is your preference
  
  {{{
  raster3d*
  volume
  3draster
  }}}
 
  My preference would be volume; the other two have issues. We can
 debate forever about how significant those issues are, but volume avoids
 them entirely.

 So if we decide for volume, do we plan to rename all modules like this? If
 not, we should stay with raster3d. Otherwise it will get very confusing in
 my opinion.

 {{{
 r3.info - vo.info
 r.to.rast3- t.to.volume
 }}}
 I must say I am not particularly fond of vo.* modules, but maybe it's just
 a matter of habit.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:92
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] sample vector temporal data

2014-12-03 Thread Luca Delucchi
Hi devs,

In r63363 I submitted an update of space time doc for several modules
using the new temporal dataset.

I'm going to make the new dataset available in the next days to test
and comment it.

Stay tuned ;-)

-- 
ciao
Luca

http://gis.cri.fmach.it/delucchi/
www.lucadelu.org
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by hcho):

 Replying to [comment:90 glynn]:
  Replying to [comment:88 martinl]:
 
   Let's try to focus on element list name. Under condition that we don't
 want to use abbreviated names, what is your preference
  
  {{{
  raster3d*
  volume
  3draster
  }}}
 
  My preference would be volume; the other two have issues. We can
 debate forever about how significant those issues are, but volume avoids
 them entirely.

 +1 for volume (+2 for voxel) for the exact same reason.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:93
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by martinl):

 Replying to [comment:93 hcho]:
  +1 for volume (+2 for voxel) for the exact same reason.

 -1 for voxel, than we could rename 'raster' to 'pixel' ;-)

 I am still not sure we need to rename than all modules from `r3` to
 `vo`...

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:94
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by helena):

 Replying to [comment:94 martinl]:
  Replying to [comment:93 hcho]:
   +1 for volume (+2 for voxel) for the exact same reason.
 
  -1 for voxel, than we could rename 'raster' to 'pixel' ;-)
 
  I am still not sure we need to rename than all modules from `r3` to
 `vo`...

 so is there also an agreement to rename all r3.* commands to vo.* commands
 and changing all manuals accordingly? If yes, this will require several
 weeks of focused testing and reviews/updates of all grass7 related
 manuals, tutorials and scripts.I don't have a good solution for this issue
 but I am not a big fan of changing names unless it is absolutely necessary
 or leads to a major improvement. Reading through the discussion I am
 wondering whether anybody has done a solid research on what are the 3D
 rasters referred to in different disciplines and which term is closest to
 our users backgrounds or are all these three names that are being
 considered based on what each individual thinks is right? Which brings me
 to another point - is this being discussed on the users list as well? The
 users will be the ones most affected (and potentially confused) so why not
 get some feedback from them?

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:95
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by martinl):

 Replying to [comment:95 helena]:
   -1 for voxel, than we could rename 'raster' to 'pixel' ;-)
  
   I am still not sure we need to rename than all modules from `r3` to
 `vo`...
 
  so is there also an agreement to rename all r3.* commands to vo.*
 commands and changing all manuals accordingly? If yes, this will require
 several weeks of focused testing and

 not at all, personally I don't like to stick with shortened element names
 (rast, rast3d) just because it makes our life easier. Taking in account a
 real world it seems to me that the reasonable is to rename `rast -
 raster` and `rast3d - raster3d` and to modify the parser to understand
 also shortened element names.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:96
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] Individual vector feature processing: v.clump?

2014-12-03 Thread Markus Metz
On Wed, Dec 3, 2014 at 5:51 PM, Markus Neteler nete...@osgeo.org wrote:
 Hi,

 for a calculation I need to loop over all vector features in a map in
 order to perform individually a computation. For that I cannot use the
 category information since it may be used as 1:n link.

 I wonder if a new v.clump command would make sense or the addition of
 a new method in v.category to simplify this task?

Adding unique categories to a new layer for the features in question
with v.category (take care of the type option) does not work in this
case?

Markus M

 Eventually I would implement the processing loop as a Python script.

 Markus
 ___
 grass-dev mailing list
 grass-dev@lists.osgeo.org
 http://lists.osgeo.org/mailman/listinfo/grass-dev
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] [GRASS GIS] #2503: wxdigit: wrong undo redo

2014-12-03 Thread GRASS GIS
#2503: wxdigit: wrong undo  redo
-+--
 Reporter:  mlennert |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  wxGUI| Version:  svn-trunk
 Keywords:  digitizer|Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by mmetz):

 Replying to [comment:8 mlennert]:
  Replying to [comment:7 mmetz]:
   The list of updated lines as returned by the vector lib was
 incomplete. Fixed in r63349,50 (trunk, relbr70), please test.
 
  Still not quite:
 [http://tomahawk.ulb.ac.be/moritz/wxdigit_undo_redo4.ogv].
 
  If you need more explanation, I can write some, but right now I have to
 go.

 Got it. This bug (yet another one) should be fixed in r63364,5 (trunk
 only).

-- 
Ticket URL: https://trac.osgeo.org/grass/ticket/2503#comment:9
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by cmbarton):

 Can raster_3D be better parsed in long and shortened form?

 Michael

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:97
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] [GRASS GIS] #2409: last call for options keys consolidation

2014-12-03 Thread GRASS GIS
#2409: last call for options keys consolidation
--+-
 Reporter:  martinl   |   Owner:  grass-dev@…  
 Type:  task  |  Status:  new  
 Priority:  blocker   |   Milestone:  7.0.0
Component:  Default   | Version:  unspecified  
 Keywords:  standardized options  |Platform:  Unspecified  
  Cpu:  Unspecified   |  
--+-

Comment(by annakrat):

 Replying to [comment:97 cmbarton]:
  Can raster_3D be better parsed in long and shortened form?
 
 It has the same issues as raster3d, still rast is ambiguous. You can
 shorten raster_3D to rast3d or r3 but you can't shorten raster.

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2409#comment:98
GRASS GIS http://grass.osgeo.org

___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

Re: [GRASS-dev] Individual vector feature processing: v.clump?

2014-12-03 Thread Markus Neteler
On Wed, Dec 3, 2014 at 9:47 PM, Markus Metz
markus.metz.gisw...@gmail.com wrote:
 On Wed, Dec 3, 2014 at 5:51 PM, Markus Neteler nete...@osgeo.org wrote:
 Hi,

 for a calculation I need to loop over all vector features in a map in
 order to perform individually a computation. For that I cannot use the
 category information since it may be used as 1:n link.

 I wonder if a new v.clump command would make sense or the addition of
 a new method in v.category to simplify this task?

 Adding unique categories to a new layer for the features in question
 with v.category (take care of the type option) does not work in this
 case?

Likely it does (I am asking on behalf on another person not subscribed here).
I wonder how to make it more obvious.  And, if there is a best
practice example written in Python/PyGRASS?

Markus
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] addons dependencies

2014-12-03 Thread Markus Neteler
On Wed, Dec 3, 2014 at 7:17 PM, Martin Landa landa.mar...@gmail.com wrote:
 2014-12-03 19:07 GMT+01:00 Moritz Lennert mlenn...@club.worldonline.be:
 v.in.wfs2 url= http://geoserver.gis.irisnet.be/ows layers=urbis:URB_A_MZ
 output=urbis srs=31370
 Downloading data from WFS server...
 Reprojecting data...
 Importing vector map into GRASS...
 WARNING: Please update the interface of the module: option dsn has been
 renamed to input

 please try out r63362.

 WARNING: Please update the user interface of v.in.ogr: option dsn
 has been renamed to input


To me that sounds a bit misleading since we don't need to update
v.in.ogr. How about

WARNING: Please update the usage of v.in.ogr: option dsn has been
renamed to input
?

Markus
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev