Re: [GRASS-dev] [GRASS GIS] #1825: r.tileset only returns one tile

2014-01-26 Thread GRASS GIS
#1825: r.tileset only returns one tile
-+--
 Reporter:  pvanbosgeo   |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  7.0.0
Component:  Raster   | Version:  svn-trunk
 Keywords:  r.tileset|Platform:  Unspecified  
  Cpu:  Unspecified  |  
-+--

Comment(by neteler):

 Proposed patch submitted in r58782. Please test (and close ticket if issue
 is now fixed).

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/1825#comment:4
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] #2152: cd command does not work in GUI Command console

2014-01-26 Thread GRASS GIS
#2152: cd command does not work in GUI Command console
-+--
 Reporter:  wenzeslaus   |   Owner:  grass-dev@…  
 Type:  defect   |  Status:  new  
 Priority:  normal   |   Milestone:  6.4.4
Component:  wxGUI| Version:  6.4.3
 Keywords:  cd, command console  |Platform:  All  
  Cpu:  Unspecified  |  
-+--

Comment(by neteler):

 Replying to [comment:2 wenzeslaus]:
  Replying to [comment:1 neteler]:
 ...
  I actually like the patch for the parser, I wanted propose
  `--h` and inclusion of `--help` in the manual page myself.

 Yes, the patch adds this, too (run make distclean to see it).

  I just don't like the wording of the `--help` description,

 Submitted with improved wording in r58783 (GRASS 7 only so far).

-- 
Ticket URL: http://trac.osgeo.org/grass/ticket/2152#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] Handling of Python scripts on MS Windows

2014-01-26 Thread Benjamin Ducke
On 25/01/14 16:48, Markus Metz wrote:
 On Sat, Jan 25, 2014 at 3:59 PM, Helena Mitasova hmit...@ncsu.edu wrote:
 Just a note, given that most of these problems were caused by conflicts with 
 python installed by ArcGIS,
 I checked with our GIS support and the latest ArcGIS 10.2 installs python2.7 
 which (I guess) should work with GRASS.
 
 No, there are different versions of Python 2.7, and not all work with
 GRASS, see e.g. ticket 2015


 On Jan 24, 2014, at 3:03 PM, Glynn Clements wrote:


 Markus Metz wrote:

 Where it gets problematic is if the user already has a Python
 installation but it's not suitable for whatever reason. In the worst
 case they may be faced with a choice between using GRASS or using
 whatever the existing Python was installed for.


There is a tool called Virtualenv that seems to address exactly
this problem: https://pypi.python.org/pypi/virtualenv

See also here: http://docs.python-guide.org/en/latest/starting/install/win/

I found it via this thread:
http://stackoverflow.com/questions/1108974/switch-versions-of-python

 IMHO keeping in mind that many GIS-interested winGRASS-users may already
 have been installed other (GIS-)software including a system-wide python
 installation, that will be the demanding challenge to provide a suitable
 solution ...

I don't know in how far a pre-existing system installation of
Python would interfere with Virtualenv or similar solutions.

But I hope that the above could be a useful approach to
solving this problem.

Best,

Ben


 How many users will have versions which are:

 a) too old for GRASS to use, and
 b) required to be that old by the existing package?

 Bear in mind that GRASS won't be the only package affected by an
 outdated system-wide Python installation.
 
 GRASS should not use a system-wide Python installation, or more
 precisely, should explicitly ignore any system-wide python file
 associations. Expecting system-wide python file associations is the
 cause of all the trouble.
 
 The embedded but not installed Python version coming GRASS works fine
 and should IMHO be used for scripts via
 
 @%GRASS_PYTHON% %GISBASE%/scripts/SCRIPT_NAME %*
 

 AFAIK, the primary case where another package requires a system-wide
 Python installation is ArcGIS, no? IIRC, ArcGIS requires Python 2.6;
 is there some fundamental reason why this isn't suitable for GRASS? If
 so, does ArcGIS require that exact version, or can it use a later
 version?
 
 We can not care for all cases of other software (versions) relying on
 a system-wide python installation.

 yes, given that most of these problems were caused by conflicts with python 
 installed by ArcGIS,
 I checked with our GIS support and the latest ArcGIS 10.2 installs python2.7 
 which should work with GRASS
 (correct me if I am wrong).
 We had similar situation with ArcGIS9* and GRASS6.3 where you had to use a 
 specific winpython2.5 build 212
 to have both of them working on the same machine.
 The test suggested by Markus in the related message would be still useful,
 but upgrading to ArcGIS10.2 should solve the problem?
 
 You can hardly suggest users to upgrade ArcGIS if they want to use GRASS...
 

 Therefore I would suggest to keep using the embedded Python version
 which is known to work

 Actually, it known not to work, hence this thread.
 
 It works as long as system-wide python file associations are ignored.
 Other Python versions might not work.
 

 The only way you can make execution of Python scripts seamless (i.e.
 works with system(), subprocess.Popen(), etc) is for the .py extension
 to be associated with a suitable interpreter (or launcher) in the
 registry.
 
 I disagree. For example, shell scripts work just fine in GRASS 6.4,
 even though there is no associated suitable interpreter (or launcher)
 in the registry.
 

 Any other approach will trap us into an endless maintenance burden,
 identifying cases where we have to provide special handling for Python
 scripts then implementing that handling.
 
 We did so for shell scripts, AFAIK this works.
 
 Markus M
 ___
 grass-dev mailing list
 grass-dev@lists.osgeo.org
 http://lists.osgeo.org/mailman/listinfo/grass-dev
 



-- 
Dr. Benjamin Ducke, M.A.
{*} Geospatial Consultant
{*} GIS Developer

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


Re: [GRASS-dev] Handling of Python scripts on MS Windows

2014-01-26 Thread Glynn Clements

Markus Metz wrote:

 On Sat, Jan 25, 2014 at 3:59 PM, Helena Mitasova hmit...@ncsu.edu wrote:
  Just a note, given that most of these problems were caused by conflicts 
  with python installed by ArcGIS,
  I checked with our GIS support and the latest ArcGIS 10.2 installs 
  python2.7 which (I guess) should work with GRASS.
 
 No, there are different versions of Python 2.7, and not all work with
 GRASS, see e.g. ticket 2015

Any version of Python 2.7 should be suitable for GRASS. However, it
needs to be installed correctly, and GRASS needs to not attempt to
bundle Python.

 GRASS should not use a system-wide Python installation, or more
 precisely, should explicitly ignore any system-wide python file
 associations. Expecting system-wide python file associations is the
 cause of all the trouble.

Executing a script uses the registry associations for the script's
extension.

Attempting to by-pass the system's script execution mechanism (by
explicitly executing Python scripts via a specific interpreter) is the
cause of all the trouble.

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


[GRASS-dev] [GRASS GIS] #2173: Tcl NVIZ: load state file Error: must be surf_id set_mask_mode BOOLEAN

2014-01-26 Thread GRASS GIS
#2173: Tcl NVIZ: load state file Error: must be surf_id set_mask_mode BOOLEAN
+---
 Reporter:  hamish  |   Owner:  grass-dev@…  
 Type:  defect  |  Status:  new  
 Priority:  major   |   Milestone:  6.4.4
Component:  Tcl/Tk NVIZ | Version:  svn-develbranch6 
 Keywords:  load state, raster  |Platform:  Linux
  Cpu:  x86-64  |  
+---
 Hi,

 there is a bug in writing the saved-state file in tcl/tk NVIZ. It means
 that re-loading a state file with raster layers leads to an inconsistent
 internal state.

 The reason is the mask-map inversion status being written as 00 not 0.
 Fixing the file by hand solves the issue and lets you load the state file
 correctly.

 spearfish example:
 {{{
 GRASS g.region -d
 GRASS nviz elev=elevation.dem

File -- Save State
Save as test.nviz

 GRASS head -n 27 test.nviz | tail -n 17
 surf*1390794941
 map elevation.dem
 0
 map elevation.dem
 0
 unset
 00
 unset
 const 60.00
 unset
 #88
 0.00 0.00 0.00
 3 3
 2 2
 poly
 grid_surf
 gouraud


 GRASS nviz state=test.nviz

 Loading raster map elevation.dem@PERMANENT...
  100%
 Loading raster map elevation.dem@PERMANENT...
  100%
 Translating colors from raster map elevation.dem@PERMANENT...
  100%
 Diagnostic: Error: must be surf_id set_mask_mode BOOLEAN --
   Load procedure for panel start surf may not be defined
 }}}

 And the internal state becomes a bit messed up: The surface renders but
 the Raster Surfaces controls show none loaded.

 In map_obj.c set_mask_mode() fails because it is saved in the file as 00
 not 0 (7th line of the 'surf' entry).

 The code that creates it is get_mask_mode() in
 visualization/nviz/src/map_obs.c, but that is using `sprintf(tmp, %d,
 mode);` to create the string so I'm not seeing where/how the 00 is being
 introduced, since %d should lead to just 0.
 ?

 The work-around is to edit the .nviz state file by hand and change the
 00 to 0. Then the Load State works as expected.

 Interestingly the first time I launch tcl nviz in 6.4.3 on a different
 machine it wrote out 0 but later attempts following the exact same
 series of clicks writes out 00.. smashed stack?

 see also scripts/panel_surf.tcl for the reading and writing of the
 session-state file (Nviz_surf_save() is well commented).


 Another question is if after fixing the source of the 00 trouble if we
 allow sloppy-read, i.e. to auto-translate 00 to 0 upon load so the user
 can load old+broken .nviz state files created by this bug. Or as a general
 policy if keeping work-arounds in the code base for old bugs generates too
 much code-clutter? In this case I would suggest to err on the side of
 being kind to the victims.


 thanks,
 Hamish

-- 
Ticket URL: https://trac.osgeo.org/grass/ticket/2173
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] #2173: Tcl NVIZ: load state file Error: must be surf_id set_mask_mode BOOLEAN

2014-01-26 Thread GRASS GIS
#2173: Tcl NVIZ: load state file Error: must be surf_id set_mask_mode BOOLEAN
+---
 Reporter:  hamish  |   Owner:  grass-dev@…  
 Type:  defect  |  Status:  new  
 Priority:  major   |   Milestone:  6.5.0
Component:  Tcl/Tk NVIZ | Version:  svn-develbranch6 
 Keywords:  load state, raster  |Platform:  Linux
  Cpu:  x86-64  |  
+---
Changes (by hamish):

  * milestone:  6.4.4 = 6.5.0


Comment:

 Replying to [ticket:2173 hamish]:
  Interestingly the first time I launch tcl nviz in 6.4.3 on a
  different machine it wrote out 0 but later attempts following
  the exact same series of clicks writes out 00.. smashed stack?

 sorry, my mistake. That happened when switching between 6.4.3 and 6.5, so
 the bug is in 6.5 only.


 Hamish

-- 
Ticket URL: https://trac.osgeo.org/grass/ticket/2173#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

[GRASS-dev] change linear back to bilinear in grass7 ?

2014-01-26 Thread Helena Mitasova
As we are going through the class material I have noticed that in GRASS7
in r.resample.interp method bilinear was changed to linear and bicubic was 
changed to cubic.

I would like to ask whether it would be OK to change it back for two reasons:
- consistency with 6.4 (minimizes changes required in tutorials, scripts, 
assignments etc.)
- bilinear interpolation is not linear, (see the relevant text in wikipedia  
you need to scroll down to find the relevant section)

Also, the change to linear and cubic has created additional inconsistencies in 
manual pages of
r.resamp.bspline
v.sample
v.drape
which have bicubic or bilinear in the examples and the text, but cubic and 
linear in the command SYNOPSIS parameters.

Helena


Helena Mitasova
Associate Professor
Department of Marine, Earth, and Atmospheric Sciences
2800 Faucette Drive, Rm. 1125 Jordan Hall
North Carolina State University
Raleigh, NC 27695-8208
hmit...@ncsu.edu

All electronic mail messages in connection with State business which are sent 
to or received by this account are subject to the NC Public Records Law and may 
be disclosed to third parties.” 

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


[GRASS-dev] [GRASS GIS] #2174: Tcl NVIZ: load state loads then forgets the look here focus position

2014-01-26 Thread GRASS GIS
#2174: Tcl NVIZ: load state loads then forgets the look here focus position
---+
 Reporter:  hamish |   Owner:  grass-dev@…  
 Type:  defect |  Status:  new  
 Priority:  normal |   Milestone:  6.4.4
Component:  Tcl/Tk NVIZ| Version:  svn-develbranch6 
 Keywords:  load state, focus  |Platform:  Linux
  Cpu:  x86-64 |  
---+
 Hi,

 In Tcl/Tk NVIZ if you change the map's location on the canvas with the
 Look Here button or change from the puck to the center view-adjust
 method, then 'File - Save State', the resulting position is successfully
 saved to the session-state file by Nget_focus(), and is loaded back in ok
 as part of panel_main.tcl's Nviz_main_load()'s Nset_focus(). But then the
 focus position gets clobbered (twice) by panel_surf.tcl and panel_vol.c
 calling look_center, which resets the view. On a less important note
 those extra two look_center mean that the screen gets redrawn a couple
 extra times too; minor performance hit.

 Commenting out the two look_center() at the ends of the Nviz_surf_load()
 and Nviz_vol_load() tcl functions lets it reload the correct original view
 upon File - Load State.


 I hesitate to commit any change without some sort of check that the focus
 has already been loaded/set, maybe those two look_center() are there for
 the first layer to be loaded when starting from a blank canvas? Note that
 AFAIK the canvas center focus is based on g.region not the bounds of the
 newly loaded surface/volume, so the view should not have to be recentered
 because a change of layer.


 thanks,
 Hamish

-- 
Ticket URL: https://trac.osgeo.org/grass/ticket/2174
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] change linear back to bilinear in grass7 ?

2014-01-26 Thread Martin Landa
Dear Helena,

2014-01-27 Helena Mitasova hmit...@ncsu.edu:
 As we are going through the class material I have noticed that in GRASS7
 in r.resample.interp method bilinear was changed to linear and bicubic was 
 changed to cubic.

for the record, there was a discussion about that a while ago [1]. So
probably we will to open this discussion again.

Martin

[1] http://lists.osgeo.org/pipermail/grass-dev/2013-May/063855.html
___
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev