-
Notifications
You must be signed in to change notification settings - Fork 1
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
Validation keywords IACS #867
Comments
Un truc bizarre déjà c'est l'URL https://inspire.ec.europa.eu/theme/lc/lc n'existe pas. https://inspire.ec.europa.eu/theme/lc/lc.fre.xml échoue alors que https://inspire.ec.europa.eu/theme/lc.fre.xml Ce n'est peut être pas la cause de l'erreur - difficile de vérifier sans installer le validateur et le modifier. |
Le code source du test doit être https://github.com/inspire-eu-validation/ets-repository/blob/v2023.3/metadata/2.0/datasets-and-series/ets-md-iacs-bsxets.xml#L61 Le message est "Land cover INSPIRE data theme is not declared for LPIS data." alors qu'on a dans la fiche <gmd:MD_Keywords>
<gmd:keyword>
<gmx:Anchor xlink:href="http://inspire.ec.europa.eu/theme/lc">Occupation des terres</gmx:Anchor>
</gmd:keyword> Nous sommes sur ce message d'erreur Donc dans ces tests let $invalidLPIS :=
if((count(distinct-values($keywordCharacterString[.=$lpisLabels]))) > 0
and (count(distinct-values($keywordCharacterString[.=$lcLabels])) = 0))then
local:addMessage('TR.dataThemeNotDeclared', map {'filename': local:filename($record), 'featureType':local-name($record), 'gmlid': string($record/@*:id), 'theme': 'Land cover', 'property': 'LPIS'})
else if(count(index-of($keywordAnchor, 'http://inspire.ec.europa.eu/metadata-codelist/IACSData/lpis')) > 0
and (count(index-of($keywordAnchor, 'https://www.eionet.europa.eu/gemet/en/inspire-theme/lc')) = 0
and count(index-of($keywordAnchor, 'http://inspire.ec.europa.eu/theme/lc')) = 0
and count(index-of($keywordAnchor, 'https://www.eionet.europa.eu/gemet/en/concept/4612')) = 0))
then
local:addMessage('TR.dataThemeNotDeclared', map {'filename': local:filename($record), 'featureType':local-name($record), 'gmlid': string($record/@*:id), 'theme': 'Land cover', 'property': 'LPIS'})
else () Le premier
Donc dans notre fiche on a bien http://inspire.ec.europa.eu/theme/lc mais pas http://inspire.ec.europa.eu/metadata-codelist/IACSData/lpis - c'est pas ça le problème ? |
Réponse obtenue sur le GitHub du validateur, c'est tout une question d'ancres (outre le mot clé libre identifié par François) |
TODO: Add anchor encoding for IACS vocabulary |
and GEMET http://geonetwork-opensource.org/gemet |
Fait. A déployer en test @GeryNi |
Je ne parviens pas à réimporter le thésaurus IACS Data (certainement moi qui ne sait pas quelle URL utiliser) J'ai l'impression que l'option anchor pour les mots clé GEMET n'est pas activée |
fonctionne pour moi. Mais j'imagine que le serveur de test n'a peut être pas les droits d'accèder aux registres INSPIRE ? A voir dans les logs @GeryNi |
donc pour GEMET oui, pour GEMET theme non. Dans l'éditeur, la présence du contrôle de l'encodage permet de savoir ce qu'il en est: A voir si on change l'encoding (mais ça veut dire migrer l'existant aussi) ? (dans un ticket dédié) |
point gemet migré ici #886 |
Le validateur ne passe tjrs pas.
Après correction manuelle de ces valeurs, ça ne passe tjrs pas. Il y a aussi des erreurs de validation de base, dont une double date de publication pour le thésaurus GEMET #870 (non résolu dans la BD de prod-temp, donc, mais semble OK dans les autres) L'encodage de la Time Extent n'est pas conforme non plus à ce qui est attendu. |
La date est fixé lors de l'import depuis le registre INSPIRE https://github.com/geonetwork/core-geonetwork/blob/main/web/src/main/webapp/xslt/services/thesaurus/registry-to-skos.xsl#L90-L92 Solutions : suppression et réimport du thésaurus (mais on a un pb de proxy pour les imports de 2nd niveau sur le registre INSPIRE).
eg. https://www.eionet.europa.eu/gemet/fr/concept/449 Donc je dirai pas de langue dans l'ancre.
Corrigé sur metawal-temp en exportant le thésaurus, correction des dates dans le fichier, suppression du thésaurus et réimport. |
Sur la fiche en question, je n'ai que des tests avec "Manual review..." L'erreur est bizarre:
|
J'ai retesté et j'avais toujours des erreurs. Après avoir cherché, j'ai trouvé les éléments suivants : Time Extent --> doit être encodé avec des beginPosition/endPosition et pas TimeInstant --> Donc : étendue temporelle > Période > Informations sur la période > Date de début/Date de fin (et pas Début/Fin) Le mot-clé IACS (SIGC) doit être d'office présent ==> l'ajouter en plus de LPIS ou GSAA. J'ai réussi à passer le test en corrigeant 2 éléments.
|
Et donc une ressource utile pour la correction de fiches en erreur, c'est les fiches types publiées sur le github du validateur |
Avec <gco:CharacterString>Common Agricultural Policy</gco:CharacterString> ou <gmx:Anchor xlink:href="http://www.eionet.europa.eu/gemet/en/concept/1600">politique agricole commune</gmx:Anchor> A suivre selon la réponse INSPIRE-MIF/helpdesk-validator#1115 |
Pris en charge par le helpdesk validator. En attente de leur réponse. Non bloquant pour la mise en prod. |
Le helpdesk validator reconnait la validité de l'encodage au niveau du concept.
Originally posted by @fabiovinci in INSPIRE-MIF/helpdesk-validator#1115 (comment) |
Problème avec le validateur ou avec les fiches ? |
Essayé ce matin avec le validateur en ligne, ça fonctionne. Même erreur que Vincent avec le validateur MW Il reste cependant des erreurs dans le contenu, au niveau de la date de publication du thesaurus IACS data et il ne détecte toujours pas les mots clés INSPIREet Gemet |
Réessayé ce soir avec le validateur en ligne. Pour la fiche https://metawal.wallonie.be/geonetwork/inspire/fre/csw?REQUEST=GetRecordById&SERVICE=CSW&version=2.0.2&elementSetName=full&outputSchema=http://www.isotc211.org/2005/gmd&id=7ba2cb90-a5d0-46f6-9502-481a25951e55 L'erreur sur ce mot-clé est en correction côté validateur INSPIRE, par contre le commit n'a pas l'air de corriger qqch. D'après ce que je vois, il a juste indiqué l'ancre avec https et avec http mais sans retirer la langue - inspire-eu-validation/ets-repository@fec7f64 Je les ai interpellés sur ce point. inspire-eu-validation/ets-repository#855 (comment) |
@marceaulouis La date du thésaurus IACS a été corrigée. Est-ce que tu as édité puis sauvé la fiche pour mettre à jour le thésaurus avant de tester ? |
Les mots clé INSPIRE LC ne sont pas reconnus, pourtant selon moi il est encodé de la même façon que LU dans la fiche 7ba2cb90-a5d0-46f6-9502-481a25951e55 https://metawal.wallonie.be/geonetwork/srv/fre/catalog.search#/metadata/e0df2619-0999-45f6-9f6c-3d241415c624, idem pour https://metawal.wallonie.be/geonetwork/srv/fre/catalog.search#/metadata/d752fab9-4560-4a32-9da6-227b51fca867 |
@marceaulouis J'ai corrigé sur https://metawal.wallonie.be/geonetwork/inspire/fre/catalog.search#/metadata/e0df2619-0999-45f6-9f6c-3d241415c624 et le test passe |
Pour réponse à François sur INSPIRE-MIF/helpdesk-validator#1115 (comment) Je n'arrive pas à tester en staging : via MW, j'ai une erreur Via le validateur en ligne version staging, je ne passe pas l'épreuve de vérification qui est buggée chez moi A suivre |
J'ai des erreurs avec le validateur IACS (version en ligne), au niveau des keywords
J'ai l'impression que le validateur ne gère pas le multilingue, mais avant de faire un ticket sur leur GitHub et passer pour une nouille, est-ce que vous pourriez jeter un œil ? Je loupe p-e une évidence
fiche testée : https://metawal.wallonie.be/geonetwork/srv/eng/catalog.search#/metadata/314690bf-016c-4f07-a6d2-de894ba8020b
csw : https://metawal.wallonie.be/geonetwork/inspire/fre/csw?REQUEST=GetRecordById&SERVICE=CSW&version=2.0.2&elementSetName=full&outputSchema=http://www.isotc211.org/2005/gmd&id=314690bf-016c-4f07-a6d2-de894ba8020b
résultat du test : https://inspire.ec.europa.eu/validator/test-run/details.html?id=EIDd768a28d-2c30-4083-9028-e0df82aa201d
The text was updated successfully, but these errors were encountered: