Error on apply qml style for a raster layer in 3.4.1 and Master 0ea86bf0a5

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)
Original Redmine Issue: 20409
Affected QGIS version: 3.4.1
Redmine category:symbology

---

New description:

After loading a saved QML style, this changes to a different one when clicking on "apply". Data, style and screencast attached.

Old description:
in 2.18.25 it works fine with the same data and qml
in 3.4.1 and Master 0ea86bf it don't works
I attach a screencast, data and qml file

---

• qml_style.7z (Antonio Viscomi)
• qml_style2.avi (Antonio Viscomi)
• sl_raster_rec_350.qml (Antonio Viscomi)

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

was the qml created in 2.18?

---

• status_id was changed from Open to Feedback

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Giovanni Manghi wrote:

was the qml created in 2.18?

yes
I also noticed that, when I load a qml in 3.4.1 an Master (regardless of whether it was created with 3.4, 3.5 or 2.x) ,
"visualization type" It automatically switch on "palette"
even if the style had been saved in pseudo color mode

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

I also noticed that, when I load a qml in 3.4.1 an Master (regardless of whether it was created with 3.4, 3.5 or 2.x) ,
"visualization type" It automatically switch on "palette"
even if the style had been saved in pseudo color mode

this is a different issue (I think) please file a different ticket (with sample data and style), thanks.

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

I also noticed that, when I load a qml in 3.4.1 an Master (regardless of whether it was created with 3.4, 3.5 or 2.x) ,
"visualization type" It automatically switch on "palette"
even if the style had been saved in pseudo color mode

just tried this in qgis master and can't confirm.

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

• regression was changed from 1 to 0
• operating_system was changed from windows 7 to
• description was changed from in 2.18.25 it works fine with the same data and qml
  in 3.4.1 and Master 0ea86bf it don't works
  I attach a screencast, data and qml file to New description:

After loading a saved QML style, this changes to a different one when clicking on "apply". Data, style and screencast attached.

Old description:
in 2.18.25 it works fine with the same data and qml
in 3.4.1 and Master 0ea86bf it don't works
I attach a screencast, data and qml file

• status_id was changed from Feedback to Open
• priority_id was changed from High to Normal

[qgib]

Author Name: Martin Dobias (@wonder-sk)

---

I think this a consequence of removal of the "Load" button to load min/max values from raster layer properties in #3871 - it was changed to load min/max when Apply is clicked which I think here interferes with the newly loaded style (once the new min/max values are loaded, the classification is auto-updated from the current color ramp). My preference would be to have the "Load" button back so it is not tied to Apply which is somehow unexpected.

side tip: after loading QML style, you do not need to click Apply or OK. Just press Cancel button and the loaded style should stay... hard to say whether that's a bug or a feature :-)

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

Martin Dobias wrote:

I think this a consequence of removal of the "Load" button to load min/max values from raster layer properties in #3871 - it was changed to load min/max when Apply is clicked which I think here interferes with the newly loaded style (once the new min/max values are loaded, the classification is auto-updated from the current color ramp). My preference would be to have the "Load" button back so it is not tied to Apply which is somehow unexpected.

+1

side tip: after loading QML style, you do not need to click Apply or OK. Just press Cancel button and the loaded style should stay... hard to say whether that's a bug or a feature :-)

... is at the very minimum a very bad ux, borderline bug.

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Martin Dobias wrote:

side tip: after loading QML style, you do not need to click Apply or OK. Just press Cancel button and the loaded style should stay... hard to say whether that's a bug or a feature :-)

is not to contradict you but if I press close button appear what you see in the screencast named qml_style2

---

• 13715 was configured as qml_style2.avi

---

• qml_style2.avi (Antonio Viscomi)

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Giovanni Manghi wrote:

I also noticed that, when I load a qml in 3.4.1 an Master (regardless of whether it was created with 3.4, 3.5 or 2.x) ,
"visualization type" It automatically switch on "palette"
even if the style had been saved in pseudo color mode

this is a different issue (I think) please file a different ticket (with sample data and style), thanks.

I think is not a issue but the prefixed behavior for new style manager

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Giovanni Manghi wrote:

I also noticed that, when I load a qml in 3.4.1 an Master (regardless of whether it was created with 3.4, 3.5 or 2.x) ,
"visualization type" It automatically switch on "palette"
even if the style had been saved in pseudo color mode

just tried this in qgis master and can't confirm.

the screencasts was recorded on Master 8125366

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

the screencasts was recorded on Master 8125366

I'm referring to the second issue, the one that is not part of the description of this ticket (you added as a comment).

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Giovanni Manghi wrote:

the screencasts was recorded on Master 8125366

I'm referring to the second issue, the one that is not part of the description of this ticket (you added as a comment).

ah ok,
then I need to refile the second one as a new Issue?

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Antonio Viscomi wrote:

New description:

After loading a saved QML style, this changes to a different one when clicking on "apply". Data, style and screencast attached.

I attach the qml saved by Master

---

• 13716 was configured as sl_raster_rec_350.qml

---

• sl_raster_rec_350.qml (Antonio Viscomi)

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

I attach the qml saved by Master

is not a matter of qml, explanation has been give here in #28229 (comment)

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

ah ok,
then I need to refile the second one as a new Issue?

you sure about it? I can't confirm here.

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Giovanni Manghi wrote:

ah ok,
then I need to refile the second one as a new Issue?

you sure about it? I can't confirm here.

Yes, I do
Because in screencast qml_style2 it appear this... But I'm not sure It is an issue

[qgib]

Author Name: Giovanni Manghi (@gioman)

---

Yes, I do
Because in screencast qml_style2 it appear this... But I'm not sure It is an issue

the only issue I see (in your cast) is that when loading the style it is applied correclty it does not switch to "palletted". If you close the properties and reopen them it shows correctly "palletted", is it?

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

Giovanni Manghi wrote:

Yes, I do
Because in screencast qml_style2 it appear this... But I'm not sure It is an issue

the only issue I see (in your cast) is that when loading the style it is applied correclty it does not switch to "palletted". If you close the properties and reopen them it shows correctly "palletted", is it?

Yes, even if the style is saved by 3.5 in pseudocolor mode when you load it or open it again It automatically switches to palette

[qgib]

Author Name: Antonio Viscomi (@Antoviscomi)

---

this issue appears even in the case of a reclassification of a raster making vain the same reclassification