-
Notifications
You must be signed in to change notification settings - Fork 24
Calf ration implementation #2043
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
Open
JoeWaddell
wants to merge
41
commits into
dev
Choose a base branch
from
calf_ration_implement
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
b6dcdb0
base logic for non-user-defined implementation
JoeWaddell 50f72d7
logic for average ration
JoeWaddell bdca2d7
fixed my wonky logic
JoeWaddell e43f7f7
added initial udr logic for calves
JoeWaddell f6b1c16
Merge bdca2d7e8ca4c04312411447f86f15039cf42869 into c5502ffc9a837b8a5…
JoeWaddell e0d6b34
Apply Black Formatting
github-actions[bot] 857ba16
scaling fix
JoeWaddell 8356be1
Update badges on README
JoeWaddell 1e72e0f
Merge branches 'calf_ration_implement' and 'calf_ration_implement' of…
JoeWaddell 793320e
Merge 1e72e0f23750566d765f7e9b0177f8b12a76e4ba into c5502ffc9a837b8a5…
JoeWaddell bbb6376
Apply Black Formatting
github-actions[bot] 6e00516
fixing errors
JoeWaddell 717c8ff
Merge branch 'calf_ration_implement' of https://github.com/RuminantFa…
JoeWaddell 0cd4890
Merge 717c8ff4acffff19d7feb71a0dbf2a8522a98fcf into c5502ffc9a837b8a5…
JoeWaddell c26b0e1
Apply Black Formatting
github-actions[bot] 3331d03
Update badges on README
JoeWaddell 7051294
Update calf_ration.py
tomhuhh e246839
updating calf replacer and starter intakes
KFosterReed 872ae9f
updating calf replacer and starter intakes
KFosterReed 7de14a6
Merge remote-tracking branch 'origin/calf_ration_implement' into calf…
KFosterReed 2761db1
Merge remote-tracking branch 'origin/calf_ration_implement' into calf…
KFosterReed bdbd8f7
Merge remote-tracking branch 'origin/calf_ration_implement' into calf…
KFosterReed d66211b
Fix typo in calf UDR method
KFosterReed eed9fed
Merge branch 'dev' into calf_ration_implement
JoeWaddell 15a331a
Merge eed9fed9ba71955c131b56a3a615d50424ddb7c8 into 12be8a0296138a0c2…
JoeWaddell 498d610
Apply Black Formatting
github-actions[bot] ca218d3
Update badges on README
JoeWaddell 1136a43
added calf nutrient requirements to animal
JoeWaddell cf2f5c1
added handle_calf_ration to pen
JoeWaddell 3e9dcc3
added methods to calf ration manager
JoeWaddell 7edc11f
Merge branch 'calf_ration_implement' of https://github.com/RuminantFa…
JoeWaddell 46d03f0
added udr logic to handle calf ration
JoeWaddell a5f4da1
updated get_user_defined_ration to only use dry matter intake instead…
JoeWaddell 6472529
updated make_ration_from_user_values for calves
JoeWaddell aeb4fc7
Merge branch 'dev' into calf_ration_implement
JoeWaddell 5c8c92e
Merge aeb4fc731e4eeb4bfef0f14be9a2411d3aecde67 into d31a0ca0260fb9fbd…
JoeWaddell 789e488
Apply Black Formatting
github-actions[bot] 2c6ea1b
Update badges on README
JoeWaddell 378575a
added elif
JoeWaddell a8fa3b1
Merge 378575a9588bf0b9b1c12df090f2371304bf40c1 into d31a0ca0260fb9fbd…
JoeWaddell b0fb53e
Apply Black Formatting
github-actions[bot] File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this If-else statement eliminate the need for the user input for
"milk_type"? If so, can we delete it while keeping the model backwards compatible for input files that have this input? Or, maybe we just make an issue about user input adjustments that need to be made?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct! The intent here was that it could streamline the feed inputs a little bit...with the user simply selecting the feed IDs for calves as they do for the other animal classes, since it's currently a little redundant.
We could still keep this input, but we'd have to think through the logic of when (or if) it should ever override the user defined feeds.
If we do deprecate it: to maintain backwards compatibility (especially with FARM ES) what I thought is that we could keep the option in the metadata, and simply not let it get to the codebase itself. I think we could/should capture it and flag a warning that the input has been deprecated, and the user should specify the feeds in the feed input JSON (we may want to think about a process for any input field deprecation we do in the future).