Closed midraed closed 9 years ago
Se vinculó un sólo commit pero hay varios con los cambios. Ya debería estar todo lo necesario para GeoINTA.
Excelente, ahora cuando @sbanchero pueda, seguimos por ese lado!
Bueno.. para los que no sabian, hace mas de un mes que se pueden ver los perfiles de SisINTA en geoINTA. Hay que elegir un proyecto que se llama "perfiles sisINTA" en el visor.
Lo que no funciona es la actualizacion? Porque me parece que esta congelado desde la primera vez que tomo datos? @sbanchero @mauriciopasquier ?
Estimados, si, me falta terminar una parte de la rutina de actualización. Lo tengo encolado al issue ese. @midraed, paciencia en breve lo resuelvo.
El 12 de noviembre de 2014, 17:38, Guillermo Federico Olmedo < notifications@github.com> escribió:
Bueno.. para los que no sabian, hace mas de un mes que se pueden ver los perfiles de SisINTA en geoINTA. Hay que elegir un proyecto que se llama "perfiles sisINTA" en el visor.
Lo que no funciona es la actualizacion? Porque me parece que esta congelado desde la primera vez que tomo datos? @sbanchero https://github.com/sbanchero @mauriciopasquier https://github.com/mauriciopasquier ?
— Reply to this email directly or view it on GitHub https://github.com/INTA-Suelos/SiSinta/issues/66#issuecomment-62789074.
Santiago Banchero
No hay problema!! Era por si no estaban al tanto!
Cuando esté eso cerramos el issue? O queda algo que hacer de mi parte @sbanchero ?
Esto es una de las caras de la moneda de #35
Estimado @mauriciopasquier https://github.com/mauriciopasquier creo que no queda nada de tu parte. Yo estoy ultimando detalles. Entre hoy y mañana ya lo dejo andando.
El 13 de noviembre de 2014, 19:24, Mauricio Pasquier Juan < notifications@github.com> escribió:
Cuando esté eso cerramos el issue? O queda algo que hacer de mi parte @sbanchero https://github.com/sbanchero ?
— Reply to this email directly or view it on GitHub https://github.com/INTA-Suelos/SiSinta/issues/66#issuecomment-62977831.
Santiago Banchero
On Mon, Nov 17, 2014 at 09:37:04AM -0800, Santiago Banchero wrote:
Estimado @mauriciopasquier https://github.com/mauriciopasquier creo que no queda nada de tu parte. Yo estoy ultimando detalles. Entre hoy y mañana ya lo dejo andando.
groso!
@mauriciopasquier, @midraed:
quedó andando, corre a las 9:30, 12:30 y 16:30...
Da de alta los nuevos perfiles y actualiza si se modificó alguno, loggea todos los cambios y le agregué una fecha de update a la tabla original así cuando se modifica algún registro se guarda también.
Saludos Santiago
2014-11-17 14:50 GMT-03:00 Mauricio Pasquier Juan notifications@github.com :
On Mon, Nov 17, 2014 at 09:37:04AM -0800, Santiago Banchero wrote:
Estimado @mauriciopasquier https://github.com/mauriciopasquier creo que no queda nada de tu parte. Yo estoy ultimando detalles. Entre hoy y mañana ya lo dejo andando.
groso!
— Reply to this email directly or view it on GitHub https://github.com/INTA-Suelos/SiSinta/issues/66#issuecomment-63344838.
Santiago Banchero
@sbanchero fijate que cuando haces la consulta en el visor de geointa sobre un perfil muestra la URL pero no está el vinculo con la página, y tampoco deja copiar la URL.
Ahí está corregido. Hay un link que creo que está, por ahora, medio al pedo que es del perfil. Ya que solo sirve si uno está autenticado, en la db me parece bien que lo guardemos pero mostrarlo en esa lista me parece que no aporta nada. ¿Opiniones?
El 19 de noviembre de 2014, 5:23, Marcos Angelini notifications@github.com escribió:
@sbanchero https://github.com/sbanchero fijate que cuando haces la consulta en el visor de geointa sobre un perfil muestra la URL pero no está el vinculo con la página, y tampoco deja copiar la URL.
— Reply to this email directly or view it on GitHub https://github.com/INTA-Suelos/SiSinta/issues/66#issuecomment-63606221.
Santiago Banchero
On Wed, Nov 19, 2014 at 04:57:46AM -0800, Santiago Banchero wrote:
Ahí está corregido. Hay un link que creo que está, por ahora, medio al pedo que es del perfil. Ya que solo sirve si uno está autenticado, en la db me parece bien que lo guardemos pero mostrarlo en esa lista me parece que no aporta nada. ¿Opiniones?
los perfiles se pueden marcar como públicos, con lo que no requieren autenticación algunos.. yo dejaría el link
Si... creo que hay que dejar el link al perfil. Porque:
1) cuando este el sisitema de permisos... luego del click y la ventana de "no tiene permisos" uno podria pedir permisos. 2) EVantualmente... la mayoria de los perfiles que estan cargados va a tener que pasar a ser publicos, dado que son datos viejos.. y muchos publicados 3) Si me parece que podriamos pasar en el geojson si tiene datos publicos o privados y mostrar con estilo diferente en geoINTA. 4) Y me voy un poco de tema... pero estaria bueno tener como una valoracion del perfil que este penalizada si el perfil es privado.. tipo red social.. puedo ponerlo en otro issue... pero seria un motivador para hacerlo publico.. 5) si ya lo hemos hablado.. algo ocn Santiago tambien de que en el estado actual.. no es muy util la informacion... y la responsabilidad de hacerla util es nuestra!
6) @sbanchero en la ventana de query del visualizador de geointa.. podriamos mostrar un grafico? Porque podriamos generar un grafiquito.. como el de USDA. Y mostrarlo ahi... Estoy hablando de un futuro no cercano de todas maneras..! ( @angelini75 no tengo el grafico que me refiero a mano.. vos lo tenes? )
Aca esta la imagen..!
cierro este issue porque está andando. Los puntos que decís para el futuro y lo del gráfico pueden ir en sus propios issues. Los movés @midraed ?
Lo del gráfico era mas bien algo para geoINTA, aunque se relaciona con #36 (donde podria haber un grafico contextualizando.. aunque meditemoslo) y con #1 (que se puede usar para generarlos, aprovechando la integracion de sisINTA con aqp de R)
Si me parece que falta, y por eso abro de nuevo: Deberiamos pasarle tambien a GeoINTA que perfiles son publicos, para que @sbanchero pueda mostrarlos de forma diferente. Dado que como hablamos antes: que cada click te lleve a un mensaje de que no tenes acceso, no es útil para nadie.
De paso y perdón por irme por las ramas, lo mismo tenemos que hacer en el visualizador de la portada nuestra. Aunque aca sería bueno, no que muestre cuales son publico, sino a cuales tenes acceso (que es diferente cuando estas logueado). Lo anoto en #35.
Yo cerré porque la funcionalidad más o menos está, y los cambios sobre esa base (como pasar qué perfiles son públicos) pueden ir en un issue aparte, así no hacemos issues kilométricos que nunca podemos cerrar (no se ve el avance!)
De acuerdo!
Con respecto a la integración con geoINTA, y según lo que hablamos con @mauriciopasquier y @sbanchero en CyA, en marzo pasado:
geoINTA lee el geojson una vez por día.
Actualmente el geojson tiene los siguientes atributos:
El acuerdo fue, y para que la información fuera más útil:
Como comparación, cuando uno ingresa a sisINTA sin estar logueado, puede ver la lista de perfiles, y el valor de: fecha, nombre:serie, ubicación, número y si es modal; quizás podriamos pensar en incluir acá tambien la clase... pero eso lo dejamos para la #36