search

datagov-cz / sgov-assembly-line

Issue tracker k výrobní lince sémantických konceptuálních modelů.
MIT License
3 stars 0 forks source link

Nekonzistentni model pro termin ma kapacitu #59

Closed blcham closed 2 years ago

blcham commented 2 years ago

Pokud je to možné, pokuste se informaci o chybě rozepsat dle následující struktury, pomůžete nám ji rychleji identifikovat a opravit.

Popis problému

kapacita_509534051675524364

Závažnost problému

Alternativy řešení

MichalMed commented 2 years ago

Miro, já nejsem autorem slovníku ani zmíněného pojmu. Asi bysme měli vymyslet nějaký workflow, jak řešit tyto nekonzistence. Jestli přes GH assignout autora slovníku, nebo to jen upravit. Problém je, že nevíme, kde všude je pojem použit a jak.

MichalMed commented 2 years ago

Při pohledu do dat v SSP tam typ-vlastnosti nikde není. V ontographeru jsem změnil stereotyp na typ vztahu (tos mohl udělat taky) a vypadá to, že je to +/- opraveno. Množství je dítě @jakubklimek , takže případné stížnosti by měly jít za ním, případně je upravím v modelu, až se dohodnete.

MichalMed commented 2 years ago

BTW Miro, tys tam dělal nějaký změny? Chtěl jsem vypublikovat tu změnu stereotypu pouze na typ-vlastnosti a je tam toho evidentně mnohem víc, než jen tohle. Jestlis tam dělal změny ty, tak prosím zreviduj tento PR: https://github.com/opendata-mvcr/ssp/pull/313

blcham commented 2 years ago

Miro, já nejsem autorem slovníku ani zmíněného pojmu. Asi bysme měli vymyslet nějaký workflow, jak řešit tyto nekonzistence. Jestli přes GH assignout autora slovníku, nebo to jen upravit. Asignovani na Tebe je automaticke, vzdyt ja jsem navrhnoval ze Vam to klidne nareportuji jak doted pres SSP na konkretni osobu ... no chtel si to takhle.

Jestli trvas natom, ze to chces reportovat tady ?! tak mne nenapada nic lepsiho jako: 1) pridat navic cloveka ktereho se to tyka 2) alternativne udelat primo pull request v tom jinom projekte a referencovat ho tady (to klidne budu delat rovnou)

@psiotwo

Problém je, že nevíme, kde všude je pojem použit a jak.

Tak muzeme se podivat sem abychom zjistili, kde je to pouzite: https://xn--slovnk-7va.gov.cz/datov%C3%BD/turistick%C3%A9-c%C3%ADle/pojem/kapacita

blcham commented 2 years ago

BTW Miro, tys tam dělal nějaký změny? Chtěl jsem vypublikovat tu změnu stereotypu pouze na typ-vlastnosti a je tam toho evidentně mnohem víc, než jen tohle. Jestlis tam dělal změny ty, tak prosím zreviduj tento PR: opendata-mvcr/ssp#313

psiotwo commented 2 years ago

@MichalMed v tomto repozitáři jsi historicky default assignee všech ticketů - myšlenka byla (a nepamatuju si, že bychom ji revidovali), že tento repozitář bude sběrný a bude existovat jeden člověk (teď ty), který ho bude mít v hlavě a bude rozhazovat na ostatní, to co je třeba. Jestli z tohoto procesu cokoliv nefunguje, nebo má fungovat jinak je to IMO námět na retro.

MichalMed commented 2 years ago

Podle mě to platí pro všechny tickety týkající se vývoje. Tohle do toho moc nezapadá. Tady jsou nějaký špatný asi data, ale to asi není předmětem bug/feature developmentu. Ten workflow by určitě bylo dobrý probrat, ostatně jak píšu ve vlákně

MichalMed commented 2 years ago

Jinak já jsem na ry data kouknul, upravil a udělal PR. Ten obsahuje mnohem více změn nez kolik jsem udělal, tak budu muset někde vyšťourat kdo to byl. Btw ten typ vlastnosti NIKDE není

MichalMed commented 2 years ago

OG by měl mít kontrolu konzistence UFO (i OntoUML) podle pravidel daných vztaženými stereotypy. Bude to celkově docela dost pravidel, takže bych šel postupně.

  1. UFO stereotyp má každý pojem jenom jeden, přidáním nového ztrácí pojem původní.
  2. Dědičnost mezi nimi by asi měla fungovat podobně (@blcham - pokud se nepletu, tak typ objektu nemůže být generalizací typu vlastnosti apod.),
  3. Dědičnosti mezi OntoUML vztahy (Kind nemůže specializovat roli atd...)
  4. OG by měl tato pravidla kontrolovat i u slovníků, které přijdou odjinud a nějak se zachovat (buď jeden apriori zrušit, nebo vyhazovat někde v OG warning, nebo poslat mail gestorovi slovníku...)