[Qgis-developer] New GRASS plugin: a test drive

Pedro VenĂ¢ncio pedrongvenancio at gmail.com
Thu Oct 22 16:41:42 PDT 2015


Hi Radim,


> Yes, it is now intended for two reasons. First, it could be confusing
> for user if some maps were missing from input list. Second is
> technical, to get list of layers in vector map may be time consuming,
> the info must be read from the map (which was not the case with < 2.11
> where only layers in canvas were listed).
>
> All vector maps are listed in map input widget, but only the types
> specified by typemask will be offered for selection (if there are more
> than one) under the input widget. If selected vector does not contain
> any layer of specified type, a warning dialog opens when module is
> run.
>
>
Right, I think it's ok this way!



> > Another question, I would like to include v.to.3d in QGIS-GRASS. To do
> that,
> > I separated it in three different modules: v.to.3d.fixed (fixed height
> for
> > 3D vector features), v.to.3d.attr (attribute column used for height) and
> > v.to.2d (reverse transformation). Do you think this can be merged now,
> > during feature freeze, since it is nothing that can corrupt anything, or
> > should I make a pull request now and you check and merge after the end of
> > feature freeze?
>
> In general it is new feature, but in this little amount, I think that
> it can be accepted, it cannot break anything, if default.qgm is a
> valid xml. In any case, test well before you create pull request and
> after it gets merged.
>
>

I've tested in several ways, and I think it works:

https://github.com/qgis/QGIS/pull/2393


Thanks again Radim!

Pedro



> Radim
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-developer/attachments/20151023/2895a8e0/attachment.html>


More information about the Qgis-developer mailing list