[GRASS-dev] [GRASS GIS] #307: v.in.dxf: output name should be required

#307: v.in.dxf: output name should be required
----------------------+-----------------------------------------------------
Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
     Type: defect | Status: new
Priority: major | Milestone: 6.4.0
Component: Vector | Version: svn-develbranch6
Keywords: | Platform: All
      Cpu: All |
----------------------+-----------------------------------------------------
All import modules require an output map name. Why is v.in.dxf and
exception and has it optional?

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

#307: v.in.dxf: output name should be required
-----------------------+----------------------------------------------------
  Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.4.0
Component: Vector | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: All | Cpu: All
-----------------------+----------------------------------------------------
Changes (by martinl):

* cc: martinl (added)

Comment:

Name of output vector map can be determined from input file name, e.g.
file.dxf become 'file' vector map if output parameter is not given.

It was changed in GRASS7 where output is required. I would suggest leave
v.in.dxf in GRASS6 as it is.

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

#307: v.in.dxf: output name should be required
-----------------------+----------------------------------------------------
  Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
      Type: defect | Status: new
  Priority: major | Milestone: 6.4.0
Component: Vector | Version: svn-develbranch6
Resolution: | Keywords:
  Platform: All | Cpu: All
-----------------------+----------------------------------------------------
Comment (by glynn):

Replying to [comment:1 martinl]:
> Name of output vector map can be determined from input file name, e.g.
file.dxf become 'file' vector map if output parameter is not given.
>
> It was changed in GRASS7 where output is required. I would suggest leave
v.in.dxf in GRASS6 as it is.

Agree re: GRASS6, just for compatibility's sake.

For 7.x, I have removed this kind of "magic" behaviour whenever I have
encountered it. Apart from anything else, it means that the overwrite
check has to be performed manually, rather than letting G_parser() do it.

It's worse for modules which create new maps (import and analysis). E.g.
it might be desired to add an option to the GUI to automatically add
newly-created maps to the map display. Obviously that can only work if the
GUI knows that a map has been created, and what it's called.

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

#307: v.in.dxf: output name should be required
--------------------------+-------------------------------------------------
  Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
      Type: enhancement | Status: new
  Priority: major | Milestone: 6.4.0
Component: Vector | Version: svn-develbranch6
Resolution: | Keywords: import
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Changes (by hamish):

  * keywords: => import
  * type: defect => enhancement

Comment:

v.in.ogr can do the same -- it will automatically set the output name from
the layer name if it is not given. IMO it is a really nice time (and typo)
saving feature and I'd be sad to see it go.

Hamish

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

#307: v.in.dxf: output name should be required
--------------------------+-------------------------------------------------
  Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
      Type: enhancement | Status: new
  Priority: trivial | Milestone: 6.4.0
Component: Vector | Version: svn-develbranch6
Resolution: | Keywords: import
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Changes (by hamish):

  * priority: major => trivial

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

#307: v.in.dxf: output name should be required
--------------------------+-------------------------------------------------
  Reporter: msieczka | Owner: grass-dev@lists.osgeo.org
      Type: enhancement | Status: new
  Priority: trivial | Milestone: 7.0.0
Component: Vector | Version: svn-trunk
Resolution: | Keywords: import
  Platform: All | Cpu: All
--------------------------+-------------------------------------------------
Changes (by martinl):

  * version: svn-develbranch6 => svn-trunk
  * milestone: 6.4.0 => 7.0.0

Comment:

Replying to [comment:3 hamish]:
> v.in.ogr can do the same -- it will automatically set the output name
from the layer name if it is not given. IMO it is a really nice time (and
typo) saving feature and I'd be sad to see it go.

Do you think to make 'output' optional in v.in.dxf again?

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