SensingClues / sensingcluesr

R package for retrieving data from the Sensing Clues platform
GNU Affero General Public License v3.0
0 stars 0 forks source link

Package change: Get obs first, then concepts #22

Open JudithDekkers opened 6 months ago

JudithDekkers commented 6 months ago

in get_observations() worden de observaties opgesplitst dus je krijgt per observatie per concept een rij in de dataframe, zoals je ook ziet hieronder zijn er 1465 observaties maar krijg je een dataframe met 4458 rijen, dit is dus gek als je dit vergelijkt met Focus waar je 1465 ziet staan. Tot nu toe was het geen issue omdat de functie alleen in de apps werd gebruikt en daar hebben we een tabel met concepten en herstructureren we de dataframe voor de tabel met observaties, maar dit is dus eigenlijk de verkeerde volgorde want voor een gebruiker van de package is dit niet logisch en wil je per rij een observatie zien als output. En dan kun je vrij simpel dit uit elkaar trekken om per concept een rij te krijgen. Ik stel voor om dit binnen korte termijn aan te pakken, vooral als we de package op CRAN willen publiceren

df1 <- get_observations(cookie = cookie, group = "focus-project-9045965", from = Sys.Date()-30, to = Sys.Date()-1) Start downloading observations ... Expecting total 1465 results in 30 number of pages Successfully processed observations in 48 seconds dim(df1) [1] 4458 12

This ticket is to change the package

JudithDekkers commented 6 months ago

after this ticket is implement, we have to implement SensingClues/wildcat-api-r#342