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

Add "needs soil functional variability" parameter to soil ID list #70

Open
paulschreiber opened this issue Apr 22, 2024 · 1 comment
Open

Comments

@paulschreiber
Copy link
Member

paulschreiber commented Apr 22, 2024

Add a "needs soil functional variability" parameter to soil ID list function. This information is only needed on site creation, not on every location lookup.

Not doing this work sill speed up the calculation.

@shrouxm
Copy link
Member

shrouxm commented May 23, 2024

i'm not sure from the perspective of the backend/mobile client that we would ever want to have this parameter be set to false? as far as i understand it won't make a huge impact on the calculation speed, but it will require the algorithm to run twice. so (assuming we're caching the output of list_soils, which is seeming necessary for capri) it'd change us from a user flow something like:

  • tap location on map, wait 5s for result
  • create site and input data, wait 1s for further results
    to:
  • tap location on map, wait 4.5s for result
  • create site and input data, wait 4.5s for further results

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

No branches or pull requests

2 participants