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

#285 milk predictions #346

Closed
wants to merge 38 commits into from
Closed

#285 milk predictions #346

wants to merge 38 commits into from

Conversation

erwinspeybroeck
Copy link
Collaborator

Changed accordingly the remarks

erwinspeybroeck and others added 8 commits October 27, 2022 11:28
for the moment only the milk url
I introduced a new type
adapted url's to reflect meilk predictions
the tag get-existing-milking-withdrawals was not correct
@@ -0,0 +1,24 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this file.

Copy link
Collaborator

@cookeac cookeac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@erwinspeybroeck to make the changes.

@cookeac cookeac linked an issue Nov 17, 2022 that may be closed by this pull request
statistics should be part of a group
added the statistics over here
added 1.3.2 tag en attached taht to the statistics
added 1.3.2 tag and attached that to the statistics API
Copy link
Collaborator

@cookeac cookeac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@erwinspeybroeck In addition to the few suggestions I have added in the files (you should be able to just accept/commit these), please consider:

  1. Removing the changes to icarStatisticsResource and icarStatisticsGroupType as these are now corrected in the published ADE 1.3.2. Depending on how you are editing, you might choose to "stash" the files with the milk prediction changes, then refresh your branch from the Develop branch, and then put your milk prediction files back in.
  2. In the URL schemes, for the moment just leave all the tags as "-1.3-" rather than 1.3.2 or 1.4. We will update all the tags in bulk before we publish version 1.4.

Thanks

types/icarMilkingPredictionType.json Outdated Show resolved Hide resolved
types/icarMilkingPredictionType.json Outdated Show resolved Hide resolved
types/icarMilkingPredictionType.json Outdated Show resolved Hide resolved
types/icarMilkingPredictionType.json Outdated Show resolved Hide resolved
url-schemes/exampleUrlScheme.json Outdated Show resolved Hide resolved
resources/icarStatisticsResource.json Outdated Show resolved Hide resolved
url-schemes/exampleUrlScheme.json Outdated Show resolved Hide resolved
url-schemes/exampleUrlScheme.json Outdated Show resolved Hide resolved
url-schemes/exampleUrlScheme.json Outdated Show resolved Hide resolved
url-schemes/managementURLScheme.json Outdated Show resolved Hide resolved
url-schemes/managementURLScheme.json Outdated Show resolved Hide resolved
url-schemes/milkURLScheme.json Show resolved Hide resolved
url-schemes/milkURLScheme.json Outdated Show resolved Hide resolved
url-schemes/milkURLScheme.json Outdated Show resolved Hide resolved
url-schemes/managementURLScheme.json Outdated Show resolved Hide resolved
@erwinspeybroeck
Copy link
Collaborator Author

will not use it

@erwinspeybroeck erwinspeybroeck deleted the erwinspeybroeck branch March 23, 2023 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide predictions for expected milk yield
2 participants