NAVADMC / ADSM

A simulation of disease spread in livestock populations. Includes detection and containment simulation.
Other
10 stars 5 forks source link

Within Unit Prevelance toggle #963

Closed missyschoenbaum closed 4 years ago

missyschoenbaum commented 5 years ago

Just noticed, the message appears regardless of if within unit prev is turned on or turned off. WI_unit_message

ConradSelig commented 5 years ago

@missyschoenbaum We intentionally left the message static so users would still see some indication of the field existing even when the option is disabled. This is just so users are not confused as to why fields are appearing and disappearing - if you want to change the wording to be more clear that would be an easy change to make.

missyschoenbaum commented 5 years ago

Thanks for the explanation, that clears up where we are off track. Within unit prevalence doesn't show up on the Disease spread form, it shows up on the Disease progression form.

2 options when within unit prev is off: 1) Modify the message slightly to say something like "Probability of infection is NOT determined by within unit prevalence". 2) Have no message appear on Disease Spread, and instead have a message in the empty spot on Disease progression, something like "Within unit prevalence is not being used."

I will post images of where on Disease progression it appears.

missyschoenbaum commented 5 years ago

How disease progression looks with WIU prev off WI_off

How disease progression looks with WIU prev on

WI_on

missyschoenbaum commented 5 years ago

How Disease Spread looks with WIU prev off - this has extra message that doesn't make sense.

WI_off_spread How Disease Spread looks with WIU prev on - this is correct WI_on_spread

ConradSelig commented 5 years ago

How about option 3?

With within unit prevalence turned OFF: image image

With within unit prevalence turned ON: image image

missyschoenbaum commented 5 years ago

I like option 3 as well.

missyschoenbaum commented 4 years ago

Looks perfect! Glad you can spell.