-
Notifications
You must be signed in to change notification settings - Fork 111
Add option to split data into train/test/validate sets #149
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
Merged
Merged
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
e9c86aa
add new arguments and basic logic checks around them
a2989a6
initial try using np split
martham93 5165753
clearer doc strings
martham93 a1af804
fix logic
martham93 492fa92
config json new checks
martham93 072e56c
clean up
martham93 9930f5b
new config json for tests
martham93 d73858a
update tests
martham93 dd749e1
clean up test format
martham93 8f15552
fix y labels
martham93 559dfb6
fix doc string format
martham93 3eb7d32
allow for user defined number of splits
martham93 397d31b
remove length check
martham93 7df341b
remove uncessary new lines
martham93 3fcfcb7
update doc string
martham93 36855d4
exceptions instead of asserts
martham93 584e6e0
add seed to docs
martham93 d47386c
allow for more than 3 way split
martham93 ce2d7bf
updates
martham93 dd0859c
fix format
martham93 60393ff
fix formatting
martham93 37a9173
remove rint and go abck to .7, .2, .1 test train val split in testing
martham93 28e2c29
update docs for clarity
martham93 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
{"country": "portugal", | ||
"bounding_box": [ | ||
-9.4575, | ||
38.8467, | ||
-9.4510, | ||
38.8513 | ||
], | ||
"zoom": 17, | ||
"classes": [ | ||
{ "name": "Water Tower", "filter": ["==", "man_made", "water_tower"] }, | ||
{ "name": "Building", "filter": ["has", "building"] }, | ||
{ "name": "Farmland", "filter": ["==", "landuse", "farmland"] }, | ||
{ "name": "Ruins", "filter": ["==", "historic", "ruins"] }, | ||
{ "name": "Parking", "filter": ["==", "amenity", "parking"] }, | ||
{ "name": "Roads", "filter": ["has", "highway"] } | ||
], | ||
"imagery": "https://api.mapbox.com/v4/mapbox.satellite/{z}/{x}/{y}.jpg?access_token=ACCESS_TOKEN", | ||
"background_ratio": 1, | ||
"ml_type": "classification", | ||
"seed": 19, | ||
"split_names": ["train", "test", "val"], | ||
"split_vals": [0.7, 0.2, 0.1] | ||
} |
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
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.
Uh oh!
There was an error while loading. Please reload this page.