search

NAVADMC / ADSM

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

Population Import accepts ID field but fails #906

Closed missyschoenbaum closed 5 years ago

missyschoenbaum commented 5 years ago

I was testing today with the new UnitID fields. If the user make the pop file correctly, with the field name UnitID, the import works. However, if the user leaves the field name as ID, the unit_id is not filled and there's no indication that something went wrong. I'm willing to do whatever option is easiest, either fail with a message, or change the field to accept ID or UnitID.

BryanHurst commented 5 years ago

This should now accept either 'id' or 'unitid'.

missyschoenbaum commented 5 years ago

What should it accept? I will fix the documentation.

BryanHurst commented 5 years ago

Sorry, that was a phone typo. This should "now" accept either 'id' or 'unitid'

missyschoenbaum commented 5 years ago

Perfect, documentation is good then. I will run through it again.

missyschoenbaum commented 5 years ago

This looks great.