Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validaciones en la carga #82

Open
dariorodriguez opened this issue Feb 10, 2015 · 9 comments
Open

Validaciones en la carga #82

dariorodriguez opened this issue Feb 10, 2015 · 9 comments

Comments

@dariorodriguez
Copy link

Voy a ir armando acá un listado de valores posibles para los parámetros de suelo, de modo que no aparezcan datos incoherentes como pHs de 73

a) Ya que estamos vamos a empezar con los pHs: ninguno de los que aparecen en las fichas (en pasta, en H20 o en KCl) podrían tomar valores por fuera del intervalo 0-14 (aunque sea muy raro encontrar un suelo con pH 1 o 2 por ej)

@midraed
Copy link
Member

midraed commented Feb 11, 2015

Creo que lo que se peude generar es para cada campo analitico (porque estamos hablando solo de los campos analiticos) un csv con nombre_del_campo, valor_minimo_posible, valor_maximo_posible, valor_minimo_esperable, valor_maximo_esperable.

Por ejemplo (me invento los esperables)
analitico_ph_pasta, 0, 14, 3, 12

Y eso quizas podria servir para marcar datos erroneos, y datos dudosos. Incluso podria estar integrado en la visualizacion de la ficha:

ph

Si un archivo o asi te sirve @mauriciopasquier, yo puedo encargarme de armar en mi fork de desarrollo, o directamente en este, el archivo base con todos los campos, y explicarle a @dariorodriguez como editarlo e irle agregando lo valores.

Claro que a veces van a existir los valores minimos y maximos posibles y en algunos casos raros, no. Y los valores esperables.. es mas dificil, porque no siempre existen. O los suelos de uno u otro lugar pueden ser muy diferentes.

@dariorodriguez
Copy link
Author

Maravilloso. Totalmente de acuerdo

El 11 de febrero de 2015, 10:11, Guillermo Federico Olmedo <
[email protected]> escribió:

Creo que lo que se peude generar es para cada campo analitico (porque
estamos hablando solo de los campos analiticos) un csv con
nombre_del_campo, valor_minimo_posible, valor_maximo_posible,
valor_minimo_esperable, valor_maximo_esperable.

Por ejemplo (me invento los esperables)
analitico_ph_pasta, 0, 14, 3, 12

Y eso quizas podria servir para marcar datos erroneos, y datos dudosos.
Incluso podria estar integrado en la visualizacion de la ficha:

[image: ph]
https://cloud.githubusercontent.com/assets/2235494/6147404/ed1746ec-b1d5-11e4-8a5e-81f70f1a6636.png

Si un archivo o asi te sirve @mauriciopasquier
https://github.com/mauriciopasquier, yo puedo encargarme de armar en mi
fork de desarrollo, o directamente en este, el archivo base con todos los
campos, y explicarle a @dariorodriguez https://github.com/dariorodriguez
como editarlo e irle agregando lo valores.

Claro que a veces van a existir los valores minimos y maximos posibles y
en algunos casos raros, no. Y los valores esperables.. es mas dificil,
porque no siempre existen. O los suelos de uno u otro lugar pueden ser muy
diferentes.


Reply to this email directly or view it on GitHub
#82 (comment).

Darío

@mauriciopasquier
Copy link
Member

@midraed si querés ir armandolo así, buenísimo. Pero dudo que el csv sea la forma final en la que se use si son valores que no van a cambiar (los mínimos y máximos al menos). Sobre los esperados, ustedes sabrán si vale la pena. Se puede usar lo que haya y si no hay ignorar ese chequeo.

@midraed
Copy link
Member

midraed commented Feb 11, 2015

@mauriciopasquier no, creo que no son valores que vayan a cambiar, una vez acordemos. Si puede pasar que en el futuro se rediscuta algo, y se mueva un poco un valor esperable.. pero no deberia ser algo comun y que por lo tnato necesite facil actualizacion.
Lo del csv, es para que te sea mas facil tomarlo, vs. tomarlo del texto de la discusion del issue. Pero pensandolo bien te serviria mas en una semilla, como el formato de las ayudas? Tipo:

- id: 1
campo: analitico_ph_pasta
min_pos: 0
max_pos: 14
min_esp: 3
max_esp: 12

o es directamente algo que meterias en el codigo, y de cualquier manera te viene bien porque va a terminar en copy paste?
Avisame que te viene mejor.

Con respecto a los valores esperables, si, deberian ser opcionales.

Y ahora pregunta de git/github:
Lo mas sencillo seria hacer ese archivo en INTA-Suelos/SiSinta: desarrollo, pero si queres lo hago en midraed/SiSinta: desarrollo y te hago un pull request. Pero si el que va a editar es dario.. con solo ponerlo de colaborador en mi repo alcanza?
Tambien podemos armar otr branch en INTA-Suelos... pero me cuesta decidir que es lo mas correcto.
Dario podria hacer un fork.. pero lo vamos a complicar al cuete.

@mauriciopasquier
Copy link
Member

Para mí es algo que va a terminar en el código (por las validaciones, por ejemplo) de una u otra forma. Asique no me preocuparía por el formato en el que lo arman, lo veo más como documentación por ahora. Por la misma razón, no creo que sea necesario meterlo como archivo en el repositorio. Tal vez sea mejor hacer una tabla acá en el wiki: https://github.com/INTA-Suelos/SiSinta/wiki/Valores-posibles

@midraed
Copy link
Member

midraed commented Feb 11, 2015

Ok.. excelente. Ahí agregué toda la lista de campos analiticos. @dariorodriguez te animas a
completarla?
Tenes que apretar edit, y luego poner los valores entre las barritas. Cualquier cosa: me decis!

@dariorodriguez
Copy link
Author

Esto te va a costar un par de birras

El 11 de febrero de 2015, 11:27, Guillermo Federico Olmedo <
[email protected]> escribió:

Ok.. excelente. Ahí agregué toda la lista de campos analiticos.
@dariorodriguez https://github.com/dariorodriguez te animas a
completarla?
Tenes que apretar edit, y luego poner los valores entre las barritas.
Cualquier cosa: me decis!


Reply to this email directly or view it on GitHub
#82 (comment).

Darío

@midraed
Copy link
Member

midraed commented Feb 11, 2015

no hay problema.

@dariorodriguez
Copy link
Author

Bueno...eso ya lo completé
Consulté a Daniel Carreira para datos de laboratorio, a Patricia Carfagno para valores de constantes hídricas y Romina Romaniuk me dió una mano con algunos valores que tenía dudas (consultó a la gente del laboratorio de suelos de Edafo-UBA)
Creo que ya está listo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants