Re: [QGIS-Developer] Processing Split with lines algorithm broken

[Anita Graser]

On Wed, Nov 15, 2017 at 6:09 AM, Nathan Woodrow  wrote:

> IMO go with the principle of least astonishment.  I think the expected
> outcome is single part features as it's not clear that you need to run
> another step.
>

​+1 Anita​
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] Processing Split with lines algorithm broken

[Nathan Woodrow]

IMO go with the principle of least astonishment.  I think the expected
outcome is single part features as it's not clear that you need to run
another step.

- Nathan

On Wed, Nov 15, 2017 at 2:48 PM, Nyall Dawson 
wrote:

> On 15 November 2017 at 00:33, matteo  wrote:
> > Hi devs,
> >
> > the algorithm Split with lines seems not working. No output error but
> > both lines or polygons in output are not split. The same algorithm with
> > the same data works in QGIS 2.18
>
> I suspect it's working OK, but you're hitting a change in 3.0 where
> the output of this algorithm is a multi-line, with the individual
> split parts still forming a single feature. You need to run multiparts
> to singleparts on it afterward.
>
> However
>
> This was a change I requested in
> https://github.com/qgis/QGIS/pull/3798, since it seemed like a logical
> move. Having used this algorithm extensively since (and also been
> confused in thinking that the algorithm was broken!) I'm not convinced
> this was the right move anymore, and think that we should probably
> always output single-part features from this algorithm.
>
> What's everyone's thoughts?
>
> Nyall
> ___
> QGIS-Developer mailing list
> QGIS-Developer@lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] Processing Split with lines algorithm broken

[Nyall Dawson]

On 15 November 2017 at 00:33, matteo  wrote:
> Hi devs,
>
> the algorithm Split with lines seems not working. No output error but
> both lines or polygons in output are not split. The same algorithm with
> the same data works in QGIS 2.18

I suspect it's working OK, but you're hitting a change in 3.0 where
the output of this algorithm is a multi-line, with the individual
split parts still forming a single feature. You need to run multiparts
to singleparts on it afterward.

However

This was a change I requested in
https://github.com/qgis/QGIS/pull/3798, since it seemed like a logical
move. Having used this algorithm extensively since (and also been
confused in thinking that the algorithm was broken!) I'm not convinced
this was the right move anymore, and think that we should probably
always output single-part features from this algorithm.

What's everyone's thoughts?

Nyall
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] Merging raster-save dialog change and Windows workflow

[Tisham Dhar]

The first section is sans WLS . Yes WLS makes everything easier and is what I 
will use going forward.

-Original Message-
From: Nyall Dawson [mailto:nyall.daw...@gmail.com] 
Sent: Wednesday, November 15, 2017 11:52 AM
To: Tisham Dhar 
Cc: qgis-developer@lists.osgeo.org
Subject: Re: [QGIS-Developer] Merging raster-save dialog change and Windows 
workflow

Hi Tisham,

If you're using WLS (which I recommend), I think you can simplify this a fair 
bit. I've got a similar setup it works ok:

On 15 November 2017 at 11:02, Tisham Dhar  wrote:
> 3)  Add scripts to git-bash path

You shouldn't need this if you're running the scripts from the bash console. If 
you move your whole workflow into bash and do all your git work there it should 
work OK.

> 4)  Install Astyle binary into git-bash path somewhere

Alternatively if you turn on the "WITH_ASTYLE" option in cmake then a recent 
astyle binary will automatically be built alongside QGIS and be ready to go.

> 5)  Install activestate perl and ppm install YAML::Tiny

If you're using the WSL, you can just sudo apt-get dependencies like this.

Hope that helps! If not, I'll take a look with you if we get a spare moment on 
Friday :)

(...and if not, let's make Nathan fix it for you!)

Nyall
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] Merging raster-save dialog change and Windows workflow

[Nyall Dawson]

Hi Tisham,

If you're using WLS (which I recommend), I think you can simplify this
a fair bit. I've got a similar setup it works ok:

On 15 November 2017 at 11:02, Tisham Dhar  wrote:
> 3)  Add scripts to git-bash path

You shouldn't need this if you're running the scripts from the bash
console. If you move your whole workflow into bash and do all your git
work there it should work OK.

> 4)  Install Astyle binary into git-bash path somewhere

Alternatively if you turn on the "WITH_ASTYLE" option in cmake then a
recent astyle binary will automatically be built alongside QGIS and be
ready to go.

> 5)  Install activestate perl and ppm install YAML::Tiny

If you're using the WSL, you can just sudo apt-get dependencies like this.

Hope that helps! If not, I'll take a look with you if we get a spare
moment on Friday :)

(...and if not, let's make Nathan fix it for you!)

Nyall
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

[QGIS-Developer] Merging raster-save dialog change and Windows workflow

[Tisham Dhar]

Hi All,

I have been experimenting with pre-commit hook workflows on Windows and after a 
few iterations I have this procedure to combine building in MSVC and pushing to 
git with pre-commit hooks.


1)  Checkout as standard on windows with git-bash (line endings are an 
issue). Will have to try a fresh clone in WLS to see what difference it makes

2)  Make pre-commit hook hardlink

3)  Add scripts to git-bash path

4)  Install Astyle binary into git-bash path somewhere

5)  Install activestate perl and ppm install YAML::Tiny

6)  Commit as usual with git



1)  I also tried WLS , line ending are less of an issue here and Travis may 
be happier

2)  Install astyle 3.xx from source, stock astyle is 2.xx

3)  Install dos2unix

4)  Install cpanminus and cpanm YAML::Tiny

5)  Pre-commit hook setup should work fine from here in.

Regards,

I have a chain of un-squashed commits here - 
https://github.com/qgis/QGIS/pull/5611 . May be worth cancelling the pull and 
remaking my changes. Let me know and I will make it cleaner in the future.

Tisham Dhar
Research Engineer
Aerometrex PTY LTD
59 King William St.
Kent Town SA 5067
Australia

___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] QGIS custom widgets badly placed in Osgeo4W install

[Jürgen E . Fischer]

Hi,

On Tue, 14. Nov 2017 at 17:56:22 +0100, Jürgen E. Fischer wrote:
> Like?  Shouldn't those be covered by including
> %OSGEO4W_ROOT%\apps\%PACKAGE%\python in the PYTHONPATH.

Which BTW is what among the things the
%OSGEO4W_ROOT%\bin\python-%PACKAGENAME%.bat wrappers do.


Jürgen

-- 
Jürgen E. Fischer   norBIT GmbH Tel. +49-4931-918175-31
Dipl.-Inf. (FH) Rheinstraße 13  Fax. +49-4931-918175-50
Software Engineer   D-26506 Norden http://www.norbit.de
QGIS release manager (PSC)  GermanyIRC: jef on FreeNode


signature.asc
Description: PGP signature
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] QGIS custom widgets badly placed in Osgeo4W install

[Jürgen E . Fischer]

Hi Denis,

On Tue, 14. Nov 2017 at 14:27:41 +, Denis Rouzaud wrote:
> The file is pretty invariant, even within QGIS 2 vs 3.
> I would be in favor of placing it in place as it would solve quite some
> issues.

Like?  Shouldn't those be covered by including
%OSGEO4W_ROOT%\apps\%PACKAGE%\python in the PYTHONPATH.


Jürgen


-- 
Jürgen E. Fischer   norBIT GmbH Tel. +49-4931-918175-31
Dipl.-Inf. (FH) Rheinstraße 13  Fax. +49-4931-918175-50
Software Engineer   D-26506 Norden http://www.norbit.de
QGIS release manager (PSC)  GermanyIRC: jef on FreeNode


signature.asc
Description: PGP signature
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

[QGIS-Developer] QGIS 2.18.14 - Raster Save As to New CRS Failed

[C Hamilton]

Last night I tried to reproject a raster layer from a New York State CRS to
EPSG:3857 by right mouse clicking on the layer, selecting "Save As", and
specifying EPSG:3857 CRS. The resulting layer it created had invalid data
in it. I tried QGIS 2.14 and it worked fine.

Has this error already been reported?

Thanks,

Calvin
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

[QGIS-Developer] Processing Split with lines algorithm broken

[matteo]

Hi devs,

the algorithm Split with lines seems not working. No output error but
both lines or polygons in output are not split. The same algorithm with
the same data works in QGIS 2.18

Should I open a ticket?

Thanks

Cheers


Matteo
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] QGIS custom widgets badly placed in Osgeo4W install

[Denis Rouzaud]

Hi Nathan,

Thanks for your reply.

The file is pretty invariant, even within QGIS 2 vs 3.
I would be in favor of placing it in place as it would solve quite some
issues.

Denis

Le mar. 14 nov. 2017 à 14:41, Nathan Woodrow  a écrit :

> Hey Denis,
>
> We can't really put it in C:\OSGeo4W\apps\Python27\Lib\site-packages\PyQt4\uic
> as it will conflict with different versions.  It would have to be in the
> installed folder for each version and then set with a custom path which is
> should already be doing.
>
> - Nathan
>
> On Tue, Nov 14, 2017 at 11:30 PM, Denis Rouzaud 
> wrote:
>
>> Hi all, mainly Jürgen I guess ;)
>>
>> During a course, I just realized qgis_customwidgets.py is badly placed
>> preventing using QGIS custom widgets within uic.loadUiType.
>>
>> It is placed in
>> C:\OSGeo4W\apps\qgis\python\PyQt4\uic
>> while it should be in
>> C:\OSGeo4W\apps\Python27\Lib\site-packages\PyQt4\uic
>>
>> Thanks a lot for fixing this :)
>>
>> Cheers,
>> Denis
>>
>> ___
>> QGIS-Developer mailing list
>> QGIS-Developer@lists.osgeo.org
>> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>>
>
>
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

Re: [QGIS-Developer] QGIS custom widgets badly placed in Osgeo4W install

[Nathan Woodrow]

Hey Denis,

We can't really put it in C:\OSGeo4W\apps\Python27\Lib\site-packages\PyQt4\uic
as it will conflict with different versions.  It would have to be in the
installed folder for each version and then set with a custom path which is
should already be doing.

- Nathan

On Tue, Nov 14, 2017 at 11:30 PM, Denis Rouzaud 
wrote:

> Hi all, mainly Jürgen I guess ;)
>
> During a course, I just realized qgis_customwidgets.py is badly placed
> preventing using QGIS custom widgets within uic.loadUiType.
>
> It is placed in
> C:\OSGeo4W\apps\qgis\python\PyQt4\uic
> while it should be in
> C:\OSGeo4W\apps\Python27\Lib\site-packages\PyQt4\uic
>
> Thanks a lot for fixing this :)
>
> Cheers,
> Denis
>
> ___
> QGIS-Developer mailing list
> QGIS-Developer@lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

[QGIS-Developer] QGIS custom widgets badly placed in Osgeo4W install

[Denis Rouzaud]

Hi all, mainly Jürgen I guess ;)

During a course, I just realized qgis_customwidgets.py is badly placed
preventing using QGIS custom widgets within uic.loadUiType.

It is placed in
C:\OSGeo4W\apps\qgis\python\PyQt4\uic
while it should be in
C:\OSGeo4W\apps\Python27\Lib\site-packages\PyQt4\uic

Thanks a lot for fixing this :)

Cheers,
Denis
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

[QGIS-Developer] Plugin [1356] ZoomToBelgium approval notification.

[noreply]

Plugin ZoomToBelgium approval by zimbogisgeek.
The plugin version "[1356] ZoomToBelgium 0.2 Experimental" is now approved
Link: http://plugins.qgis.org/plugins/ZoomToBelgium/
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer

[QGIS-Developer] Plugin [849] AmigoCloud approval notification.

[noreply]

Plugin AmigoCloud approval by zimbogisgeek.
The plugin version "[849] AmigoCloud 0.6" is now approved
Link: http://plugins.qgis.org/plugins/qgis-amigocloud-plugin/
___
QGIS-Developer mailing list
QGIS-Developer@lists.osgeo.org
List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer