Skip to content
This repository has been archived by the owner on Apr 28, 2023. It is now read-only.

educational evidences: Feedback on errata issues and deviations between tables and diagrams #124

Closed
XHochschuleDE opened this issue Oct 9, 2020 · 2 comments

Comments

@XHochschuleDE
Copy link

The German project XHEIE.EU will soon provide several consolidated comments of German Higher Education Institutions to the educational evidences on a field per field basis.

Meanwhile smaller quick-fix issues recognised during the analysis that were sent to us
(credits to Markus von der Heyde of SemaLogic UG):

  1. contains-relationship in diagramm but not in table
    https://github.com/SEMICeu/SDG-sandbox/blob/master/evidences/university_diploma_supplement_evidence/data_model/university_diploma_supplement_evidence_diagram_v0.01.png
    shows a "contains" relationship which is not shown in the corresponding table
    https://github.com/SEMICeu/SDG-sandbox/blob/master/evidences/university_diploma_supplement_evidence/data_model/university_diploma_supplement_evidence_tables_v0.01.md

  2. wrong spelling of example name in given name
    Johann Sebastian Bach the great German composer and musician of the Baroque period has its first given name spelled with two "n" (not Johan!)
    (And yes, this small error needs to be fixed also at the source meaning the core person vocabulary

  3. missing entity definition Issuing Authority in University Diploma and University Diploma Supplement and Record of Results evidence
    Issuing Authority is mentioned as entity but not defined in the model diagramm nor in the table.
    In https://github.com/SEMICeu/SDG-sandbox/blob/master/evidences/certificate_of_completion_of_secondary_education/data_model/certificate_of_completion_of_secondary_education_tables_v0.02.md the expected type was casted to Public Organisation

Sebastian commenting under the umbrella of XHochschule.de / XHEIE

@cbahim
Copy link
Collaborator

cbahim commented Oct 15, 2020

@XHochschuleDE thanks for letting us know.

  1. This will be corrected in the new version of the table.
  2. Indeed, this is an error at the Core Person Voc level. We will however change it in all our models.
  3. Issuing Authority used to be a class but is now modelled as a relationship in all models. We didn't see any definition missing.

@cbahim cbahim closed this as completed Oct 15, 2020
@SebastianSK
Copy link

SebastianSK commented Sep 22, 2021

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants