Rapporteket / norgast

R-pakke med resultatrapporter for NoRGast
0 stars 1 forks source link

Rapporteket QA gjennomgang #15

Closed HAJKU closed 3 weeks ago

HAJKU commented 3 weeks ago

Tester å legge innspill her @kevinthon Med avhakingsmulighet for gjennomført oppgave!

Har gått gjennom og sett på det vi har jobbet med av endringer. Og håper jeg har fått sett på alt :-) Det ser bra ut, men noen endringer nødvendig, og så er det noen spørsmål.

Nå får jeg opp navn istedenfor FALK-genererte brukernavn, det er flott. Ulempen er at alle brukernavnene fra før overgang til FALK har blitt borte, slik at feltene står tomme for forløp fra før januar (eller var det februar)2021. Kan det løses på noe vis? ( NB: i datadumpen «skjemaoversikt» er navnene med for alle datoer)

Her kommer brukernavn opp. Veldig fint. Men hvorfor er noen i store bokstaver, og noen i små (bagatell!)? Og så lurer jeg på hvorfor det er to navn på en del av forløpene? Ordlyden på navn på kolonnene «Opprettetav» og «sist lagret av» kunne indikere at det er kun én bruker som oppgis? Eller tar jeg feil her? Det er ikke noe problem, altså, jeg bare lurer på hvorfor det har blitt slik.

kevinthon commented 3 weeks ago

Prosessert data: De aktuelle variablene i denne datadumpen er ForstLukketAv og OppfForstLukketAv. Disse variablene ble introdusert ganske seint i registerets historie, så dataen finnes ikke lenger tilbake enn det du finner i datadump. Det har altså ikke med FALK å gjøre. Det er mulig å koble inn OpprettetAv og SistLagrtetAv fra SkjemaOversikt hvis det er viktig å ha de med også i prosessert data.

Datakvalitet: Hvorfor det er noen store og noen små bokstaver må hnikt svare på, jeg bruker bare de navnene som er der. At det er to navn er fordi basisreg og oppfølging i noen tilfeller har forskjellig registrar.

HAJKU commented 3 weeks ago

Den er god! Jeg klarer å identifisere nåværende registrarer på Ullevål/Riks, og får da rettet opp i utfordringene med feilregistreringer der. Skulle det blir et problem med historikk, så tar jeg heller opp tråden igjen.

HAJKU commented 2 weeks ago

@kevinthon jeg må legge til neo nytt her. Det er en ting jeg ikke forstår likevel. Angående brukere. Hvorfor var det slik at brukerne er med i hele historikken i "Skjemaoversikt", men ikke i Prosesserte data?

Jeg skulle gjerne hatt de der også. Sitter med saken (eller føljetongen...) fra Riksen og Ullevål, der det er en del forløp som skal flyttes fra Riksen til Ullevål. Opprinnelig fikk jeg beskjed om at alle cholecystectomier registrert på OUS skal til Ullevål. Men når jeg nøster litt, så foretas og registreres det cholecystectomier også ved Riksen, så vi må finne en måte å identifisere riktige forløp fra før OUS ble splittet (forhåpentligvis uten å måtte slå opp i EPJ for alle pasientene). Jeg håper at brukernavnene kan gi oss en pekepinn...

kevinthon commented 2 weeks ago

@HAJKU Det har å gjøre med at de eneste variablene som omhandler registrarer i AlleVarNum og ForlopsOversikt (som er grunnlag for prosessert data) er ForstLukketAv og OppfForstLukketAv. Disse variablene ble introdusert ganske seint i registerets historie, så dataen finnes ikke lenger tilbake enn det du finner i datadump. Som sagt er det fullt mulig å koble inn OpprettetAv og SistLagrtetAv fra SkjemaOversikt hvis det gjør livet ditt enklere, disse finnes for hele registerets historie.

HAJKU commented 2 weeks ago

@kevinthon Ah, da skjønner jeg. Måtte ha det inn med teskje... Jeg er usikker på hvor mye bruk det vil være for det, men akkurat i denne saken hadde det vært greit å ha det med. Hvor stor jobb er det å få på plass? Og lager det enda mer kaos i lappeteppet av instrukser du får fra oss?

kevinthon commented 2 weeks ago

@HAJKU Ny versjon er installert i QA der prosessert data har OpprettetAv og SistLagretAv merget inn fra SkjemaOversikt. Se om det ser greit ut, hvis så er det kanskje på tide å få prodsatt ny versjon? Det er vel en god del endringer som ikke har kommet i produksjon ennå...

HAJKU commented 2 weeks ago

Perfekt! Har testet. Alt ser bra ut. Prodsetting helt på sin plass!

HAJKU commented 2 weeks ago

@kevinthon