helseprofil / orgdata

:package: R package for aggregating original data mostly on enumeration areas level of granularity
https://helseprofil.github.io/orgdata/
Other
1 stars 0 forks source link

IBRUKFRA og IBRUKTIL #309

Closed vegardlysne closed 1 year ago

vegardlysne commented 2 years ago

Ønske fra Nora:

Endre settings slik at IBRUKFRA = "fra og med", og IBRUKTIL = "til", slik at nye filer blir brukt fra den datoen de innføres.

For å unngå uforutsette problemer, må dette vente til etter kubeproduksjon 2023 er ferdig.

ybkamaleri commented 2 years ago

Hvordan funker det i KHfunctions når det gjelder IBRUKFRA og IBRUKTIL? Det er sikkert lurt å gjøre det samme som brukes der for at man kan bare forholde seg lik regle enten i KHfunctions eller orgdata.

Når det gjelder IBRUKTIL så var det et ønske fra Steinar om "til og med" hvor i hans tilfelle er det "til og med idag" (#272). Kanskje det er lurt å være enige om hva som er best? @raniets

raniets commented 1 year ago

Noras opprinnelige kommentar: "Jeg har et endringsønske. Jeg vet ikke om det skal legges inn som en «issue» nå, for jeg tror jeg vil foretrekke at vi ikke gjør noe med det akkurat nå (ref. tukling). Men i orgdata er det slik at kolonnen «IBRUKTIL» for innfiler egentlig er «i bruk til og med» og det er uheldig. Jeg ønsker at det at ibrukfra er «fra og med» og ibruktil er bare «til». "

Jeg ser i KHELSA: Der har vi nå skrevet samme dato for IBRUKTIL på gammel fil og IBRUKFRA på ny fil. Det gjør det lett å se på hvilken dato man la inn den nye fila og tok ut den gamle, og det er slik Nora ønsker det for Orgdata. Og jeg ser i KHfunctions.R: Der er det et stort SQL-uttrykk som leser ut parameterne fra Access. Den delen som selekterer på dato ser slik ut: (...) WHERE INNLESING.FILGRUPPE='", filgruppe, "' AND ORIGINALFILER.IBRUKFRA<=", datef, " AND ORIGINALFILER.IBRUKTIL>", datef, " AND INNLESING.VERSJONFRA<=", datef, " AND INNLESING.VERSJONTIL>", datef, sep = "") (...) Det matcher også med hvordan Nora ønsker.

Issue 272 gjaldt egentlig ikke forskjellen mellom ">" og ">=". Det gjaldt at IBRUKTIL måtte være akkurat "01.01.9999" , ellers ble filen regnet som "ikke i bruk". En annen dato i fremtiden var ikke tilstrekkelig. Det ble endret til å sammenlikne med dagens dato, men jeg vet ikke om det er satt ">" eller ">=" som kriterium.

ybkamaleri commented 1 year ago

Det bør uansett følge det som er gjort i KHfunctions. Forstår ikke hvorfor jeg ikke har doublesjekket i KHfunctions 😞 Men siden det ikke skal innføre nå så kan det ligger i dev branch når jeg får tid å gjøre det. Når det gjelder #272 er nok min tolkning av "today" betyr "til og med".