search

VNG-Realisatie / ODS-Open-Raadsinformatie

Project repository tbv de ontwikkeling van de Open Raads- en Stateninformatie API specificatie
https://vng-realisatie.github.io/ODS-Open-Raadsinformatie/
9 stars 2 forks source link

[vraag] Bereik van "Video tijdsaanduiding aanvang" #95

Closed rien333 closed 4 months ago

rien333 commented 4 months ago

Volgens ORI heeft spreekfragment "video/audio tijdsaanduiding aanvang" en "video/audio tijdsaanduiding einde" attributen.

Bijvoorbeeld:

Attribuutsoort details Video tijdsaanduiding aanvang

Naam | Video tijdsaanduiding aanvang -- | -- Herkomst | imORI Definitie | De tijdsaanduiding naar aanvang van het SPREEKFRAGMENT in de video opname Herkomst definitie | imORI Datum opname | 1 oktober 2017 Mogelijk geen waarde | Nee Indicatie kardinaliteit | 0 .. 1 Indicatie authentiek | Authentiek Lengte | 6 Formaat | Integer Patroon | hh:mm:ss Indicatie afleidbaar |   Waarde afleidbaar | Nee

Wat voor waarde wordt er nou precies bij deze tijdsaanduidingen verwacht? Een integer, of een string in hh:mm:ss formaat? En als het een integer moet worden, wat representeert deze integer dan?

De swaggerui is niet heel veel duidelijker, helaas: 

{
  "aanwezigeDeelnemer": {
    "id": "string"
  },
   ...
  "audioTijdsaanduidingAanvang": 0,
  "audioTijdsaanduidingEinde": "string",
  "videoTijdsaanduidingAanvang": 0,
  "videoTijdsaanduidingEinde": "string",
  "positieNotulen": "string",
...
}
HenriKorver commented 4 months ago

@rien333 Bedankt voor je terechte observaties. Informatiemodel is inderdaad niet correct. Het patroon "hh:mm:ss" en de lengte "6" moeten worden verwijderd. De Video tijdsaanduiding aanvang is de tijdsaanduiding naar aanvang van het SPREEKFRAGMENT in de video opname in seconden. Bovendien moet in de definitie expliciet gemeld worden dat de tijd dus gemeten wordt in seconden.

Zelfde verhaal voor Video tijdsaanduiding einde, audioTijdsaanduidingAanvang en audioTijdsaanduidingEinde.

@melsk-r: Kun jij dit corrigeren in het IM en een nieuwe catalogus uitrollen?

In de OAS van de API (swagger, redoc) zijn "audioTijdsaanduidingEinde" en "videoTijdsaanduidingEinde" per abuis als string getypeerd, dat moet natuurlijk integer zijn. Ik zal dat corrigeren in de OAS.

HenriKorver commented 4 months ago

In de OAS van de API (swagger, redoc) zijn "audioTijdsaanduidingEinde" en "videoTijdsaanduidingEinde" per abuis als string getypeerd, dat moet natuurlijk integer zijn. Ik zal dat corrigeren in de OAS.

Gedaan! Zie redoc.

HenriKorver commented 4 months ago

Issue toch even heropend om te wachten op de correcties in het informatiemodel. Als @melsk-r daarmee klaar is kunnen we dit issue sluiten.

melsk-r commented 4 months ago

Issue, zoals Henri hierboven zegt, nog even heropend.