Loomio

3.0 Soft freeze exemptions

Nyall DawsonNyall Dawson Wed 29 Nov 2017 12:20AMPublicSeen by 24

Polls regarding whether the following PRs should be granted exemption from the soft freeze and merged prior to 3.0:

  • https://github.com/qgis/QGIS/pull/5600: "Add order by expression processing algorithm". Very low risk - it's a self contained algorithm on which nothing else depends. Processing in 3.0 is awesome, and has made great strides to becoming a competitive ETL tool. This algorithm would very nicely complement the work already done in 3.0 to make models more powerful and flexible.

  • https://github.com/qgis/QGIS/pull/5430: "More output format choices in raster save as dialog": Low risk. Changes only affect the dialog, and an extra convenience member in
    QgsRasterFileWriter (which is accompanied by unit tests)

  • https://github.com/qgis/QGIS/pull/5729: "[processing] Expose dynamic ("data defined") numeric parameters to gui". Low-medium risk. Most of the changes here are gui related, just exposing functionality which already exists in the new processing backend (and which is covered by stacks of unit tests). I'd love to see this included in 3.0 as it changes some of the processing API, and I would like wider testing and porting of algorithms to utilise this BEFORE the API is locked in place and we're stuck with it. I'd rather identify any issues in the related API here for 3.0. Additionally, the changes to the "set z value" algorithm which allows the z value to be pulled from a field or expression is extremely useful in preparing datasets for use in the new 3d views. Plus, it's an absolute KILLER feature to have.

  • https://github.com/qgis/QGIS/pull/5663: "Add a name for transactions created from executeSql". It's marked as a feature and tagged as 3.2, but looking over it it seems to be almost entirely api tweaks and a unit test. I'd say this is low risk, especially given Paul's history of stable commits and how well unit tested this are of code is already.

  • https://github.com/qgis/QGIS/pull/5922 (duplicate feature redigitized)

  • https://github.com/qgis/QGIS/pull/5925 (duplicate child feature in relationwidget of attributeform)

  • https://github.com/qgis/QGIS/pull/5961 (Contextmenu with individual actions (in form view side column))

Nyall Dawson

#5600 Add order by expression processing algorithm

poll by Nyall Dawson Closed Sat 2 Dec 2017 12:09AM

Should we merge #5600 for 3.0?

Results

ResultsOption% of pointsVoters
Yes7911Tim SuttonJürgen FischerMartin DobiasDenis RouzaudAlexander BruyNathan WoodrowNyall DawsonLarry ShafferSalvatore LarosaBorys JurgielMathieu Pellerin
Abstain213strkPaul BlottiereHugo Mercier
No00 
Undecided13Paolo CavalliniGary ShermanMatthias KuhnWerner MachoAndreas NeumannTom KralidisMagnusPirmin KalbererStéphane BrunnerAlessandro PasottiRadim BlazekSandro Manivmora

14 of 27 votes cast (51% participation)

Nyall Dawson

Merge #5430: More output format choices in raster save as dialog

poll by Nyall Dawson Closed Sat 2 Dec 2017 12:09AM

Should we merge #5430 prior to 3.0 release?

Results

ResultsOption% of pointsVoters
Yes6911Tim SuttonPaolo CavalliniMatthias KuhnJürgen FischerMartin DobiasDenis RouzaudAlexander BruyNyall DawsonLarry ShafferSalvatore LarosaMathieu Pellerin
Abstain315Andreas NeumannstrkBorys JurgielPaul BlottiereHugo Mercier
No00 
Undecided11Gary ShermanWerner MachoNathan WoodrowTom KralidisMagnusPirmin KalbererStéphane BrunnerAlessandro PasottiRadim BlazekSandro Manivmora

16 of 27 votes cast (59% participation)

Nyall Dawson

#5729 [processing] Expose dynamic ("data defined") numeric parameters to gui

poll by Nyall Dawson Closed Sat 2 Dec 2017 12:09AM

Should we merge #5729 prior to 3.0 release?

Results

ResultsOption% of pointsVoters
Yes8214Tim SuttonPaolo CavalliniMatthias KuhnJürgen FischerMartin DobiasDenis RouzaudAlexander BruyNathan WoodrowNyall DawsonLarry ShafferAndreas NeumannSalvatore LarosaBorys JurgielMathieu Pellerin
Abstain183strkPaul BlottiereHugo Mercier
No00 
Undecided10Gary ShermanWerner MachoTom KralidisMagnusPirmin KalbererStéphane BrunnerAlessandro PasottiRadim BlazekSandro Manivmora

17 of 27 votes cast (62% participation)

Matthias Kuhn

Matthias KuhnFri 1 Dec 2017 10:59AM

Yes

Having this available in the GUI will give us much more testing. And it's opt-in. +1d

Nyall Dawson

#5663: Add a name for transactions created from executeSql

poll by Nyall Dawson Closed Sat 2 Dec 2017 12:09AM

Should we merge #5663 prior to 3.0 release?

Results

ResultsOption% of pointsVoters
Yes6511Paolo CavalliniMatthias KuhnJürgen FischerMartin DobiasDenis RouzaudAlexander BruyNathan WoodrowNyall DawsonLarry ShafferAndreas Neumannstrk
Abstain244Tim SuttonSalvatore LarosaBorys JurgielMathieu Pellerin
No122Paul BlottiereHugo Mercier
Undecided10Gary ShermanWerner MachoTom KralidisMagnusPirmin KalbererStéphane BrunnerAlessandro PasottiRadim BlazekSandro Manivmora

17 of 27 votes cast (62% participation)

Tim Sutton

Duplicate feature redigitized

proposal by Tim Sutton Closed Sun 7 Jan 2018 7:48AM

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

Results

ResultsOptionVotes% of votes cast% of eligible voters
Agree66722Tim SuttonDenis RouzaudWerner MachoLarry ShafferSalvatore LarosaBorys Jurgiel
Abstain33311Paolo CavalliniAlexander BruyNyall Dawson
Disagree000 
Block000 
Undecided1867Gary ShermanMatthias KuhnJürgen FischerMartin DobiasNathan WoodrowAndreas NeumannstrkTom KralidisMathieu PellerinMagnusPaul BlottiereHugo MercierPirmin KalbererStéphane BrunnerAlessandro PasottiRadim BlazekSandro Manivmora

9 of 27 votes cast (33% participation)

Tim Sutton

Duplicate child feature in relationwidget of attributeform

proposal by Tim Sutton Closed Sun 7 Jan 2018 7:48AM

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

Results

ResultsOptionVotes% of votes cast% of eligible voters
Agree88930Tim SuttonDenis RouzaudWerner MachoAlexander BruyNyall DawsonLarry ShafferSalvatore LarosaBorys Jurgiel
Abstain1114Paolo Cavallini
Disagree000 
Block000 
Undecided1867Gary ShermanMatthias KuhnJürgen FischerMartin DobiasNathan WoodrowAndreas NeumannstrkTom KralidisMathieu PellerinMagnusPaul BlottiereHugo MercierPirmin KalbererStéphane BrunnerAlessandro PasottiRadim BlazekSandro Manivmora

9 of 27 votes cast (33% participation)

Nyall Dawson

Nyall Dawson
Agree
Thu 4 Jan 2018 8:11PM

Looks low risk to me

Tim Sutton

Contextmenu with individual actions (in form view side column)

proposal by Tim Sutton Closed Sun 7 Jan 2018 7:48AM

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

Results

ResultsOptionVotes% of votes cast% of eligible voters
Agree56319Denis RouzaudWerner MachoLarry ShafferSalvatore LarosaBorys Jurgiel
Abstain33811Paolo CavalliniAlexander BruyNyall Dawson
Disagree000 
Block000 
Undecided1970Tim SuttonGary ShermanMatthias KuhnJürgen FischerMartin DobiasNathan WoodrowAndreas NeumannstrkTom KralidisMathieu PellerinMagnusPaul BlottiereHugo MercierPirmin KalbererStéphane BrunnerAlessandro PasottiRadim BlazekSandro Manivmora

8 of 27 votes cast (29% participation)