search

OSM-Catalan / monitorOSM

Consulta canvis a OSM i genera informes de destrosses
https://osm-catalan.github.io/monitorOSM/
GNU General Public License v3.0
1 stars 4 forks source link

Create get_relation_history #2

Closed marcboschmatas closed 1 year ago

marcboschmatas commented 1 year ago

Creat funció per a obtenir la història d'una relació.

marcboschmatas commented 1 year ago

Bones, per part meva perfecte els canvis que proposes. Caldrà provar que funcioni tot.

marcboschmatas commented 1 year ago

D'altra banda, si pot agafar la història d'altres elements potser que li canviem el nom, no?

marcboschmatas commented 1 year ago

Veig que no funciona amb coses que no siguin relacions. Hi segueixo treballant i faig una nova proposta quan tingui un sistema funcional

jmaspons commented 1 year ago

Ah, tb falta pujar el fitxer .Rd

marcboschmatas commented 1 year ago

@jmaspons podries fer tu el fitxer .Rd, he treballat una mica en brut i no tinc el paquet en local.

jmaspons commented 1 year ago

Això ja fa bona pinta, @marcboschmatas!

Els errors de R CMD check i pre-commit són senzills d'arreglar. Ho vols fer tu o ho faig a posteriori?

El que m'agradaria pensar bé són els formats de sortida. Potser seria millor que cada fila de les taules fos un sol conjunt de canvis (columnes osm_id, version, changeset, timestamp, user i user_id, com a clau primària de la taula + osm_type) i que cada etiqueta fos una columna. Pels membres de les relacions i vies, el mateix. És una mica estrany, però podem posar una columna de data.frames amb les propietats dels membres.

Com ho veus? Vols provar-ho de fer tu o ho donem el PR per bo i ja afegiré els canvis més endavant?

marcboschmatas commented 1 year ago

Bones! Doncs a veure, si no corre pressa ho aniré fent jo, i ja de pas arreglo els errors del R CMD CHECK i faig la documentació i tot plegat. També anem pensant com ho integrem amb la resta de funcions, que quan detecti un canvi que no ens agrada retorni l'estat actual i l'anterior?

El dj., 22 de juny 2023, 18:12, Joan Maspons @.***> va escriure:

Això ja fa bona pinta, @marcboschmatas https://github.com/marcboschmatas !

Els errors de R CMD check i pre-commit són senzills d'arreglar. Ho vols fer tu o ho faig a posteriori?

El que m'agradaria pensar bé són els formats de sortida. Potser seria millor que cada fila de les taules fos un sol conjunt de canvis (columnes osm_id, version, changeset, timestamp, user i user_id, com a clau primària de la taula + osm_type) i que cada etiqueta fos una columna. Pels membres de les relacions i vies, el mateix. És una mica estrany, però podem posar una columna de data.frames amb les propietats dels membres.

Com ho veus? Vols provar-ho de fer tu o ho donem el PR per bo i ja afegiré els canvis més endavant?

— Reply to this email directly, view it on GitHub https://github.com/OSM-Catalan/monitorOSM/pull/2#issuecomment-1602947164, or unsubscribe https://github.com/notifications/unsubscribe-auth/APKYMO45DUGRK3Y243KAQVDXMRVF5ANCNFSM6AAAAAAZO3OQ6E . You are receiving this because you were mentioned.Message ID: @.***>

marcboschmatas commented 1 year ago

Tanco la PR que amb OsmapiR ja ho hauríem de tenir.