LayerManager / layman

Publishing geospatial vector data online
GNU General Public License v3.0
10 stars 3 forks source link

Upgrade QGIS #909

Open jirik opened 10 months ago

jirik commented 10 months ago

Trying version 3.32.2 (currently Layman uses 3.16)

So it seems QGIS 3.32.2 understands styles v3.16.

jirik commented 9 months ago

Known failing tests with 3.32.2:


        wmsi = WebMapService(wms_url, version=wms_version)
        assert publication in wmsi.contents
        wms_layer = wmsi.contents[publication]
        exp_output_srs = set(settings.LAYMAN_OUTPUT_SRS_LIST)
        assert exp_output_srs.issubset(set(wms_layer.crsOptions))
        wms_layer_bbox = next((tuple(bbox_crs[:4]) for bbox_crs in wms_layer.crs_list if bbox_crs[4] == layer_crs))
>       assert_util.assert_same_bboxes(wms_layer_bbox, layer_bbox, 0.1)

tests/static_data/single_publication/layers_test.py:248: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

bbox1 = (-180.0, -90.0, 180.0, 90.0), bbox2 = (18.913842, 48.46164, 18.913842, 48.46164), precision = 0.1

    def assert_same_bboxes(bbox1, bbox2, precision):
        assert len(bbox1) == 4, (bbox1, len(bbox1))
        assert len(bbox2) == 4, (bbox2, len(bbox2))
        for i in range(0, 3):
>           assert abs(bbox2[i] - bbox1[i]) <= precision, (bbox1, bbox2, precision, i)
E           AssertionError: ((-180.0, -90.0, 180.0, 90.0), (18.913842, 48.46164, 18.913842, 48.46164), 0.1, 0)
E           assert 198.913842 <= 0.1
E            +  where 198.913842 = abs((18.913842 - -180.0))

test_tools/assert_util.py:34: AssertionError
index-git commented 5 months ago

small_layer_334.zip Sample style small_layer.qml loaded and saved with QGIS desktop 3.34.3-Prizren ne_10m_admin_0_countries_qgis334.zip Sample style ne_10m_admin_0_countries.qml loaded and saved with QGIS desktop 3.34.3-Prizren