co-cddo / ukgov-metadata-exchange-model

A metadata model for describing data assets for exchanging between UK government organisations.
https://co-cddo.github.io/ukgov-metadata-exchange-model/
Other
13 stars 1 forks source link

Minimal examples on Distribution page result in 404 errors #59

Closed AlasdairGray closed 11 months ago

AlasdairGray commented 11 months ago

Describe the bug A 404 Not found error is returned when clicking on either the Distribution-os-postcodes-csv-distribution-minimal or the Distribution-os-postcodes-excel-distribution-minimal examples.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'https://co-cddo.github.io/ukgov-metadata-exchange-model/Distribution/#examples'
  2. Click on 'Distribution-os-postcodes-csv-distribution-minimal'
  3. See 404 Not found error

Expected behavior The JSON representation of the example should be displayed.

Screenshots Screenshot 2023-10-30 at 14 43 04

Screenshot 2023-10-30 at 14 42 34

Additional context This relates to an underlying problem in the LinkML framework that has been reported in https://github.com/linkml/linkml/issues/1508

soydaner commented 11 months ago

I could not reach the enum values as well. They returned 404 page. Are those having the same/similar issue?

https://co-cddo.github.io/ukgov-metadata-exchange-model/OrganisationValues/OrganisationValues https://co-cddo.github.io/ukgov-metadata-exchange-model/AccessRightsValues/AccessRightsValues https://co-cddo.github.io/ukgov-metadata-exchange-model/SecurityClassificationValues/SecurityClassificationValues etc.

AlasdairGray commented 11 months ago

I could not reach the enum values as well. They returned 404 page. Are those having the same/similar issue?

https://co-cddo.github.io/ukgov-metadata-exchange-model/OrganisationValues/OrganisationValues https://co-cddo.github.io/ukgov-metadata-exchange-model/AccessRightsValues/AccessRightsValues https://co-cddo.github.io/ukgov-metadata-exchange-model/SecurityClassificationValues/SecurityClassificationValues etc.

No those are not related to this issue. Each of your URLs has an extra directory on the end, they should be

soydaner commented 11 months ago

I see, that's OK. Then let me explain what happened in my case: there is a link for non existent page. I open this page: https://co-cddo.github.io/ukgov-metadata-exchange-model/OrganisationValues/ then click the URI: link and end with this page (404): https://co-cddo.github.io/ukgov-metadata-exchange-model/OrganisationValues/OrganisationValues

AlasdairGray commented 11 months ago

In order to get the examples displaying, and to work within the LinkML assumptions, we will need to change the way that we are defining the classes so that we start with the least restrictive and then increase the obligation.

This would mean that the property page would show the least restrictive usage rather than the common usage of the property. To overcome this, we propose to extract the URI value and place it a consistent position with the class pages and merge the Obligation and Cardinality table with the Applicable Classes table so that it would look something like the following

URI: accessRights Class Range Obligation Cardinality
Data Resource Uriorcurie Recommended Multiple
Distribution Uriorcurie Optional Multiple
Data Service Uriorcurie Recommended Multiple
Dataset Uriorcurie Recommended Multiple