-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
at42-getcapabilities-layer-default-style missing required styles #382
Comments
Dear @Kate-Lyndegaard , We are looking into the WMS ETS to check what we need to modify. In the meatime, could you share with us an example endpoint to test the changes? Also, if you can provide the list of layer styles it would be very much appreciated and we can have a discussion over them. Thank you |
Dear @carlospzurita, I will check with our customer to determine whether I can share an endpoint. Here is a non-exhaustive list of recommended styles, extracted from various INSPIRE data specifications that we work with:
Kind regards, |
It is quite clear that the layer names were not exhaustively checked as I noticed also this issue in the past. I think that Portrayal sections in the TGs for all data themes should be read. looking only at HH and US: According to TG the Layer names for HH are listed at page 89: According to TG the Layer names for US are listed at pages 120 & 121: The full list of codelist values (such as that PoliceService) are provided in Annex C at page 167, where first letter should be made in uppercase. There are quite many. Some are listed below: Iurie |
Dear @Kate-Lyndegaard We are modifying the ETS to cover the layer names with their default style included. As for the recommended styles, for now it is not planned to include them in the validator. This styles are collected in the data specification for each theme, and it would rather complex to keeping them updated. Also, is noteworthy the TG section on this topic As it is a recommendation, we are not checking it on the validator. |
Dear @carlospzurita, That sounds great. As we invest a lot of effort in maintaining the recommended styles, it would've been nice to have them recognized, however I understand your point:-) I'm glad that the default styles can be added. Kind regards, |
PS.ProtectedSitesArchaeological, PS.ProtectedSitesCultural, PS.ProtectedSitesLandscape and many others related to Protected Sites should be also added. Please see #491 |
This is the list of harmonised layer names that are missing for Protected Sites Data Theme: Based on https://inspire.ec.europa.eu/codelist/Natura2000DesignationValue: Based on https://inspire.ec.europa.eu/codelist/IUCNDesignationValue: Based on https://inspire.ec.europa.eu/codelist/DesignationSchemeValue: Based on https://inspire.ec.europa.eu/enumeration/ProtectionClassificationValue: |
I think that actually therea are two issues that are in conjuntion:
|
Dear all, we relaxed the related test, so now if the WMS contains only harmonised layer names the test passes otherwise it passes with a manual check. This change is already implemented in the staging instance. You can also see my comment here #39 (comment). At the moment, the validator considers only the layers defined by IR and available in the INSPIRE Registry. Please note that in some TGs the additional layers are listed together with the “official” ones - as for HH for which the TG defines an additional layer “HH.Event” to those defined by IR - and this could be lead to consider them as “official”. I hope this clarifies. |
Dear all, On request from Marco we provide feedback from Sweden on this discusson in addition to our mail response also here. We (Sweden) are of the strong opinion that a view service ALSO could contain additional layers. From our point of view we agree on your examples: “e.g. those recommended by TGs or layers related to a not-harmonized dataset”, but in addition there is also another case where it already exists layers in the service as children to the harmonized layer. In that case we would like to provide the harmonized layer as a group layer. It is important that the way of how these additional layers should be implemented is clearly defined so that there are obvious if there exists any limitations. Kind Regards Fredrik Persäter and Michael Östling |
We discussed layer naming on the thematic clusters as below link, and decided that group layering was the way to go: |
Dear @nmtoken, |
@fabiovin even in the link provided by @nmtoken the layers have names are starting with the prefix of the data theme, such as for example "GE" followed by a dot "." But to make an implementation that is allowing layer names such as "cbsprovincies" Therefore I cant really understand why you are so keen to implement the solution as proposed here #39 (comment) without looking at the text of the Directive as explained in the #39 |
Thank you for adding the style. I tested using this service: https://haleconnect.com/ows/services/org.292.504afbe6-0069-411b-b69b-2aae3728f735_wms?SERVICE=WMS&REQUEST=GetCapabilities&VERSION=1.3.0 The layer at42-getcapabilities-layer-default-style test passes, however at39-getcapabilities-layer-name fails with a null pointer error and I don't know why that would be as the layer name is valid. Test run on 08 31 - 18.04.2023 with test suite Conformance Class View Service WMS.zip Kind regards, |
Dear @Kate-Lyndegaard Thank you for your feedback. It is a strange behaviour, we will analyse it in the next few days. Regards |
Dear @Kate-Lyndegaard Could you please try again? We have done some re-configurations in order to solve a redirection issue. Regards |
I can confirm that the layer name error is no longer occurring. Test: Kind regards, |
Hi,
Could you please add
HH.HealthDeterminantMeasure.Default
andUS.EnvironmentalManagementSite.Default
to the list of tested default layer style names?We are also serving numerous "recommended styles". Have you given any thought to including the layer style names of "recommended styles" in the test? I can provide a list if that would be helpful.
Kind regards,
Kate
The text was updated successfully, but these errors were encountered: