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 Certified Milk Weight and Total Milk Weight to Milking Visit #436

Closed
cookeac opened this issue Nov 29, 2023 · 5 comments
Closed

Add Certified Milk Weight and Total Milk Weight to Milking Visit #436

cookeac opened this issue Nov 29, 2023 · 5 comments

Comments

@cookeac
Copy link
Collaborator

cookeac commented Nov 29, 2023

I'm adding this issue to allow discussion of PR #434 added by @AndreasSchultzGEA to allow discussion.
The PR adds two (not required) additional weights - certified milk weight and total milk weight.

Andreas will add an explanation of the purpose for these proposed changes.

@AndreasSchultzGEA
Copy link
Collaborator

The existing property is (in our case) normally used to transmit the certified milk-amount to a milk-recording organization. But this amount may not contain separated amounts.
On the other side, there are e.g. feeding-calculations which intend to calculate feed for animals on the complete milk-production (including separated amounts) of an animal.
Therefore, at least a second milkingMilkWeight is useful.
In addition, the third milkingMilkWeight may extend the former one containing also pre- and post-milking weights.

@cookeac
Copy link
Collaborator Author

cookeac commented Nov 30, 2023

We agreed to go back to the milk recording organisations and manufacturers and ask how milking weights (separated, discarded, certified, and total) are represented so we can get the names right.

@cookeac
Copy link
Collaborator Author

cookeac commented Mar 7, 2024

We discussed on 7 March 2024 and agreed:

  • The milkingMilkWeight within the existing icarMilkingVisitResource should remain the standard, "certified" milk weight that is used in genetic evaluations in line with the ICAR Guidelines.
  • separatedMilkWeight (for a separated quarter), discardedMilkWeight (or similar, for milk discarded during cleaning of the teat), and totalMilkWeight (the sum of these and the milkingMilkWeight) should be additional attributes
  • Our preference was to add these to the array of milkCharacteristics.

@cookeac
Copy link
Collaborator Author

cookeac commented Mar 7, 2024

We also noted that at the moment the characteristic property of icarMilkCharacteristicType is just a string defined by a description. Really this should be an enumeration. However, this could be a breaking change. We should create the enumerated type, and at the moment just link or reference this from the description of characteristic. This would greatly improve documentation.

@AndreasSchultzGEA
Copy link
Collaborator

We decided to close this item

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

No branches or pull requests

2 participants