icra / snappAPI-v2

REST API for SNAPP tool
https://snappapi-v2.icradev.cat/
MIT License
0 stars 0 forks source link

Issue55 Converteix infiltration rate the mm/s a mm/hour #64

Closed jospueyo closed 3 months ago

jospueyo commented 3 months ago

@crous-mayne, quan puguis li dones un cop d'ull i si ho veus tot bé l'aproves.

jospueyo commented 3 months ago

Hi ha un petit detall que no entenc molt bé, canvies la iinflitration rate de mm/s a mm/h. En el cas de infiltrationSoils entenc que passa de s a h amb el /3600 però el primer ho dividies per 1000 i ara per 3600 no em quadra el canvi. És correcte?

Sí, és correcte, aquí ho passo a mm/s a mm/h i a dins la funció ho passo de mm a metres. No entenc què feia abans, crec que era incorrecte, dividia body.infiltration per 1000 a fora i a dins de la funció... Sort que pràcticament ningú passa el valor, passen infiltrationSoils ja donava correcte. Ara està bé 99% segur.