KPI_7 results are 0/0 for all records

[amilan17]

Summary and Purpose
None of the 120K+ records from DWD test site appear to have graphicOverview and we should investigate the reason and determine and, if relevant, the value of keeping this KPI.
Some potential explanations that should be explored:

1. None of the records have //gmd:graphicOverview
2. The test site doesn't have a complete set of records
3. The Xpath is incorrect or too restrictive
4. There's another bug in the pwcmp

Screenshot

[amilan17]

related to #94 #123

[antje-s]

Hi Anna,

I took a look at a concrete example: urn:x-wmo:md:int.eumetsat:EO:EUM:DAT:MSG:AIRMASS

This metadata record includes text as link under gmd:graphicOverview

<gmd:graphicOverview>
  <gmd:MD_BrowseGraphic>
    <gmd:fileName>
        <gco:CharacterString>https://navigator.eumetsat.int/preview/0deg-airmass.jpg</gco:CharacterString>
    </gmd:fileName>
  </gmd:MD_BrowseGraphic>
</gmd:graphicOverview>

and KPI_7 result was:

"kpi_007": {
        "name": "KPI-7: Graphic overview for non bulletins metadata records",
        "total": 0,
        "score": 0,
        "comments": [],
        "percentage": null
    },

At the moment the code expects an Anchor under <gmd:fileName>

xpath = '//gmd:identificationInfo/gmd:MD_DataIdentification/gmd:graphicOverview/gmd:MD_BrowseGraphic/gmd:fileName/gmx:Anchor'

Example in documentation

        <gmx:Anchor xlink:href="https://navigator.eumetsat.int/preview/meteosat-msg_naturalenhncd.jpg">Meteosat MSG Natural Enhanced Color</gmx:Anchor>

And I assume that this affects several metadata sets in the same way. From the logic I also see the anchor as reasonable and better. Possibly one could be nevertheless less strict and also accept <gco:CharacterString>?

This should be discussed and decided by your team ;-)

[tomkralidis]

@wmo-im/tt-wismd for review/discussion at tomorrow's meeting.

[amilan17]

@josusky -- was this fixed in pywcmp? I think it can be fixed by removing gmx:Anchor from the Xpath. 

 //gmd:identificationInfo/gmd:MD_DataIdentification/gmd:graphicOverview/gmd:MD_BrowseGraphic/gmd:fileName

[josusky]

@amilan17 I see action in the meeting notes https://github.com/wmo-im/tt-wismd/wiki/Meeting-2022-03-17 but not a decision. Perhaps the action should have been "create pywcmp issue" :-)
(The code change will be simple)