-
Notifications
You must be signed in to change notification settings - Fork 27
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
Define enumeration for icarReproParturitionEventResource - calvingEase #185
Comments
The ICAR Interbeef Technical Manual (2015/09) defines the following 1-5 scale for calving ease:
The ICAR Dairy recording guidelines emphasise the importance of calving ease but do not provide a list.
The list in #183 is broadly compatible with the Interbeef list
IF we are to define an enumeration, I recommend the five value list from Interbeef with the possible addition of Embryotomy (removal of a dead progeny) - although this could be represented in dairy cows with Caesarean and setting the liveProgeny and totalProgeny to indicate there is a stillborn progeny. |
After our discussions today, I propose the following enumeration for **calvingEase"" (by the way, this should really be parturitionEase or birthEase to cater for other species, but too late now!): "type": "string", "enum": [ |
@cookeac I guess we don't need number prefix in values? (aka 1-EasyUnassisted -> EasyUnassisted) |
The number prefixes were for alignment with the Interbeef and other standards which document meaning but use numeric values. I would rather not have the number prefixes, but then additional documentation will be needed. |
It is better to have same naming convention for all enumeration values, I would prefer to have an additional documentation instead of prefixes :) |
Currently the calvingEase attribute of icarReproParturitionEventResource.json is a string field with no definition.
There would be benefit in a consistent enumeration or score.
Splitting this issue out from the remaining discussion in #183 so that it can be addressed specifically. This will allow further research into the other topics in that issue.
Research: Would this cause a breaking change for current implementations?
The text was updated successfully, but these errors were encountered: