-
Notifications
You must be signed in to change notification settings - Fork 0
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
Try to add a new exported state variable #15
Conversation
The bug was just because I hadn't merged in the new dust changes. Now I just need to update the tests to reflect this new state variable. |
One (optional) improvement that could be good is to find a way to suppress the warning that |
this was something we supported in odin1 and it will come back at some point (you can control how angry odin will be at you for unused things). But anything other than error forces me to think about the interface for that control, so I have paused on that for now |
Thinking ahead, I'm going to eventually want an indicator of how many outbreaks there really are, not just those declared. I tried to add this new exported state variable,
infected_herds_region
by following the guide of what was there for the others.It should just tally how many herds there are with ANY infected cows for each region.
But perhaps expectedly it's failing, could do with some insight into why.