-
Notifications
You must be signed in to change notification settings - Fork 25
[Cross Validation] Added cross validation rules for the SC module #2916
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
matthew7838
wants to merge
43
commits into
dev
Choose a base branch
from
crop-field-xvalidation
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
43 commits
Select commit
Hold shift + click to select a range
9d4f4d0
Added files
matthew7838 cf0b1f6
Updated contents
matthew7838 ed41f3f
Updated contents
matthew7838 28982d0
Updated contents in soil and field
matthew7838 73bab5e
Merge 28982d0d0f072a54694b16f611ad488f711b8a9f into 7b090b9c17b481f1c…
matthew7838 7965cbc
Apply Black Formatting
github-actions[bot] d70ad03
Added length comparison and tests
matthew7838 3d4371d
Merge branch 'dev' into crop-field-xvalidation
matthew7838 3fad6b8
Merge 3d4371d82e22a4cd178eeab872d6639687e6d023 into f794363cc15eba689…
matthew7838 c8ecd97
Apply Black Formatting
github-actions[bot] 7477e2e
Deleted extra tests
matthew7838 b3d68b0
Merge remote-tracking branch 'origin/crop-field-xvalidation' into cro…
matthew7838 4f5abc0
Merge b3d68b0b8dbb9449e6b5533d2a04e2bb9b156b86 into f794363cc15eba689…
matthew7838 359318a
Apply Black Formatting
github-actions[bot] 3a194a0
Fixed failing tests
matthew7838 c3177ca
Merge remote-tracking branch 'origin/crop-field-xvalidation' into cro…
matthew7838 330de4a
Merge c3177ca53e7b5d4e0455a2d1bf2bfe2171184176 into f794363cc15eba689…
matthew7838 9542c96
Apply Black Formatting
github-actions[bot] 567a665
Update badges on README
matthew7838 b43737b
Fixed flake8
matthew7838 822c729
Merge remote-tracking branch 'origin/crop-field-xvalidation' into cro…
matthew7838 e43e768
Updated changelog.md
matthew7838 536a8df
Merge e43e768dd5c462503406fd82c0cfd33a8f6e12f6 into f794363cc15eba689…
matthew7838 5e89527
Apply Black Formatting
github-actions[bot] c9deb91
Update badges on README
matthew7838 b42bab7
Merge branch 'dev' into crop-field-xvalidation
matthew7838 9f22729
Merge b42bab76fb40b712b817590213594a51cb46ddc0 into 472a3fefa4c4fc717…
matthew7838 1fbe613
Apply Black Formatting
github-actions[bot] 575a16e
Update badges on README
matthew7838 f3cd227
Addressed PR comments
matthew7838 a30033e
Merge f3cd2271117c8d3f806f1c8046c564c128341f10 into 472a3fefa4c4fc717…
matthew7838 c1afdf0
Apply Black Formatting
github-actions[bot] 205a646
Update badges on README
matthew7838 e946297
Removed unnecessary test
matthew7838 5746876
Merge e946297d0df1efc3780c12940a33d4732111b3d2 into 472a3fefa4c4fc717…
matthew7838 400b4bf
Apply Black Formatting
github-actions[bot] 124f810
Update badges on README
matthew7838 3586492
Removed flake8 errors
matthew7838 0ff17a8
Merge remote-tracking branch 'origin/crop-field-xvalidation' into cro…
matthew7838 d69ba2b
Merge branch 'dev' into crop-field-xvalidation
matthew7838 9c4b84e
Merge d69ba2b17c98949019c2acf59194fb2011839166 into 53349675447bf39b6…
matthew7838 516e03a
Apply Black Formatting
github-actions[bot] f691b20
Update badges on README
matthew7838 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| {"crop_schedules": | ||
| [ | ||
|
|
||
| { | ||
| "crop_species": "corn_silage", | ||
| "planting_days": [ | ||
|
|
||
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.
If this is the error message that is logged, should it be more specific? for example, "Invalid data length validation: X and Y must have equal lengths" or something.
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.
This is a great suggestion, I agree, and this issue seems like the case for most evaluation methods. However, I think we should have an issue/PR dedicated to this. I will create an issue.