INGV / qquake

A plugin for QGIS 3.x that relies on web services for loading seismological data
https://www.emidius.eu/qquake/
GNU General Public License v3.0
7 stars 3 forks source link

Missing symbology when loading extended (XML) output from fdsnws-event #71

Closed MarioLocati closed 11 months ago

MarioLocati commented 11 months ago

When in the "earthquakes parameters tabs" the symbology is not correctly assigned to epicentres leading to an apparently empty layer.

The problem occurs when the user select the "extended" output table format in conjunction with short field names.

The problem lies in the different label assigned to the field used for the magnitude, which is named:

As the field name for styling the layer is hardcoded in the loaded qml ("Magnitude"), as QGIS cannot fint such a label it cannot correctly style the epicentres leaving them transparent leaving the user puzzled because no epicentres are shown on the map even if the table contains all layers.

image

MarioLocati commented 11 months ago

These two commits https://github.com/INGV/qquake/commit/89006dea74e989947448212987a006fd2720e224 https://github.com/INGV/qquake/commit/c77004078fc5090756040c4c0460b1f976596207 should solve the problem

We should change the default selected, from the current short names to long names, at the moment I have moved up the button with https://github.com/INGV/qquake/commit/fcc6c88aed63a060bb83917afb1a07a22ec9ae0c

Now need to figure out how to set the long field names as the default, most probably here somewhere https://github.com/INGV/qquake/blob/master/qquake/gui/output_table_options_dialog.py#L65

MarioLocati commented 11 months ago

Should be solved with the following commits