Skip to content
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

Closed
Kate-Lyndegaard opened this issue Sep 7, 2020 · 18 comments
Closed
Labels
deployed in reference validator Solution deployed in production non-breaking change The change makes tests less restrictive and it has no impact on resources already tested
Milestone

Comments

@Kate-Lyndegaard
Copy link

Hi,

Could you please add HH.HealthDeterminantMeasure.Default and US.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

@project-bot project-bot bot added this to To do in Validator issues Sep 7, 2020
@carlospzurita
Copy link
Contributor

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

@Kate-Lyndegaard
Copy link
Author

Kate-Lyndegaard commented Sep 8, 2020

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:

CP.CadastralParcel.LabelOnReferencePoint
CP.CadastralParcel.ReferencePointOnly
CP.CadastralParcel.BoundariesOnly
EL.ContourLine.Master
EL.ContourLine.Ordinary
EL.ContourLine.Auxiliary
GE.GeologicUnit.Lithology
GE.GeologicUnit.AgeOfRocks
GE.GeologicFault
GE.GeologicFold
GE.GeomorphologicFeature.Natural
GE.GeomorphologicFeature.Anthropogenic
GE.Borehole
GE.Aquifer.Type
GE.Aquifer.MediaType
GE.Aquiclude.ConstitutionOfAquiclude
GE.Aquitard.ConstitutionOfAquitard
GE.AquiferSystem.ConstitutionOfAquiferSystem
GE.GroundWaterBody
GE.ActiveWell
HY.PhysicalWaters.Waterbodies.Persistence
HY.PhysicalWaters.Waterbodies.Man.Made
NZ.RiskZone
NZ.HazardArea
NZ.ObservedEvent - type of hazard
NZ.ExposedElement

Kind regards,
Kate

@iuriemaxim
Copy link

iuriemaxim commented Sep 16, 2020

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:

image

According to TG the Layer names for US are listed at pages 120 & 121:

image

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:

image

Iurie

@carlospzurita
Copy link
Contributor

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

imagen

As it is a recommendation, we are not checking it on the validator.

@project-bot project-bot bot moved this from To do to In progress in Validator issues Sep 16, 2020
@Kate-Lyndegaard
Copy link
Author

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,
Kate

@iuriemaxim
Copy link

iuriemaxim commented Jan 21, 2021

PS.ProtectedSitesArchaeological, PS.ProtectedSitesCultural, PS.ProtectedSitesLandscape and many others related to Protected Sites should be also added. Please see #491

@iuriemaxim
Copy link

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:
PS.ProtectedSitesSpecialAreaOfConservation
PS.ProtectedSitesSpecialProtectionArea
PS.ProtectedSitesSiteOfCommunityImportance
PS.ProtectedSitesProposedSpecialProtectionArea
PS.ProtectedSitesProposedSiteOfCommunityImportance

Based on https://inspire.ec.europa.eu/codelist/IUCNDesignationValue:
PS.ProtectedSitesStrictNatureReserve
PS.ProtectedSitesHabitatSpeciesManagementArea
PS.ProtectedSitesManagedResourceProtectedArea
PS.ProtectedSitesNationalPark
PS.ProtectedSitesNaturalMonument
PS.ProtectedSitesProtectedLandscapeOrSeascape
PS.ProtectedSitesWildernessArea

Based on https://inspire.ec.europa.eu/codelist/DesignationSchemeValue:
PS.ProtectedSitesIUCN
PS.ProtectedSitesNatura2000
PS.ProtectedSitesRamsar
PS.ProtectedSitesNationalMonumentsRecord
PS.ProtectedSitesUNESCOManAndBiosphereProgramme
PS.ProtectedSitesUNESCOWorldHeritage
PS.ProtectedSitesEmeraldNetwork

Based on https://inspire.ec.europa.eu/enumeration/ProtectionClassificationValue:
PS.ProtectedSitesNatureConservation
PS.ProtectedSitesArcheaological
PS.ProtectedSitesCultural
PS.ProtectedSitesEcological
PS.ProtectedSitesLandscape
PS.ProtectedSitesEnvironment
PS.ProtectedSitesGeological

@iuriemaxim
Copy link

iuriemaxim commented Jan 25, 2021

I think that actually therea are two issues that are in conjuntion:

  1. Related to harmonised layers that are missing in the Validator (i.e. layer name = PS.ProtectedSitesRamsar)
  2. Related to the harmonised styles names that are missing in the Validator. Default style for the PS.ProtectedSitesRamsar layer could be either PS.ProtectedSitesRamsar.Default or PS.ProtectedSite.Default)

@carlospzurita carlospzurita modified the milestones: v2021.0, v2021.1 Feb 9, 2021
@fabiovin
Copy link
Collaborator

fabiovin commented Mar 2, 2021

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.
The additional layers defined in the TGs are not checked but could be used by data providers.

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.

@fredper-lm
Copy link

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

@nmtoken
Copy link

nmtoken commented Mar 15, 2021

We discussed layer naming on the thematic clusters as below link, and decided that group layering was the way to go:

https://wayback.archive-it.org/12090/20161216055008/https://themes.jrc.ec.europa.eu/discussion/view/13952/layer-naming

@fabiovin
Copy link
Collaborator

fabiovin commented Mar 15, 2021

Dear @nmtoken,
thank you for sharing this discussion, I think that the solution that we are going to implement (#39 (comment)) is in line with what has been discussed.

@iuriemaxim
Copy link

iuriemaxim commented Mar 15, 2021

@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"
and styles such as "cbsprovincies:ziekenhuisopnamen_dag_per_10000_inwoners"
if at least one harmonised layer name exist in the WMS, it is quite a big difference and not sustained by any text, neither by the INSPIRE Directive.

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

@carlospzurita carlospzurita removed this from the v2021.1 milestone Apr 6, 2021
@arantzaetxebarria arantzaetxebarria removed their assignment Apr 17, 2023
@arantzaetxebarria arantzaetxebarria added ready for testing Solution provided to reporter or developed & deployed in staging (or beta), waiting for testing and removed under development labels Apr 17, 2023
@MarcoMinghini MarcoMinghini moved this from In progress to Staging: for acceptance in Validator issues Apr 17, 2023
@Kate-Lyndegaard
Copy link
Author

Hi @arantzaetxebarria,

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,
Kate

@arantzaetxebarria
Copy link
Collaborator

Dear @Kate-Lyndegaard

Thank you for your feedback. It is a strange behaviour, we will analyse it in the next few days.

Regards

@arantzaetxebarria arantzaetxebarria added under analysis and removed ready for testing Solution provided to reporter or developed & deployed in staging (or beta), waiting for testing labels Apr 19, 2023
@arantzaetxebarria
Copy link
Collaborator

Dear @Kate-Lyndegaard

Could you please try again? We have done some re-configurations in order to solve a redirection issue.
Excuse the inconvenience.

Regards

@arantzaetxebarria arantzaetxebarria added ready for testing Solution provided to reporter or developed & deployed in staging (or beta), waiting for testing and removed under analysis labels Apr 19, 2023
@Kate-Lyndegaard
Copy link
Author

Hi @arantzaetxebarria,

I can confirm that the layer name error is no longer occurring.

Test:
Test run on 08 20 - 20.04.2023 with test suite Conformance Class View Service WMS.zip

Kind regards,
Kate

@arantzaetxebarria arantzaetxebarria added solved Solution developed and accepted, not yet deployed and removed ready for testing Solution provided to reporter or developed & deployed in staging (or beta), waiting for testing labels Apr 20, 2023
@MarcoMinghini MarcoMinghini moved this from Staging: for acceptance to Staging: in release planning in Validator issues Apr 20, 2023
@fabiovinci fabiovinci added the non-breaking change The change makes tests less restrictive and it has no impact on resources already tested label Jun 21, 2023
@jenriquesoriano jenriquesoriano added deployed in reference validator Solution deployed in production and removed solved Solution developed and accepted, not yet deployed labels Jun 22, 2023
@MarcoMinghini MarcoMinghini moved this from Staging: in release planning to Production: latest release in Validator issues Jun 22, 2023
@fabiovinci fabiovinci removed this from Production: latest release in Validator issues Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed in reference validator Solution deployed in production non-breaking change The change makes tests less restrictive and it has no impact on resources already tested
Projects
None yet
Development

No branches or pull requests

10 participants