-
Notifications
You must be signed in to change notification settings - Fork 224
Dataset redesign for multi-gpu, multi-processing and multi-geometry #1057
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 24 commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
5b1273f
remove dataset from main
bw4sz 18884f5
don't overwrite utilities
bw4sz dc6e957
checking out to fix style editor
bw4sz 19b5920
seperate NMS for per-image tiles
bw4sz e34f2ef
working on global box class
bw4sz 36f2a2c
split out dataset tests
bw4sz 2e7a677
split out individual tests for each dataset strategy
bw4sz 812488e
predict_tile tests pass
bw4sz fde2f48
revert training, needs to be seperate PR
bw4sz 5c039dc
rebase main
bw4sz a7ed84f
confirm model loading once
bw4sz 1324914
local tests pass
bw4sz 006cea7
Style updates
bw4sz c9e8f90
docformatter
bw4sz 0964cc6
update tests with logging error
bw4sz 0429a53
sphinx error for deepforest datasets is now a submodule
bw4sz 8d4d8f3
more explicit multi-processing tests
bw4sz ec0e843
multiprocessing GPU error
bw4sz 41c3bfe
move the image height into __init__
bw4sz 21f095b
more multiprocessing tests
bw4sz 2c808d7
eval edits from Josh
bw4sz 9006019
improve the documentation around inference
bw4sz 90afbbd
add image on dataloader strategy
bw4sz 7543e1c
fix image path
bw4sz 97b2627
doc edits
bw4sz e5089df
predict tile paths argument
bw4sz ea6298a
Revert "predict tile paths argument"
bw4sz 7e7f1e8
remove paths argument from predict_tile
bw4sz 0bf9c72
style updates
bw4sz 84be208
avoid edge cases when predict_tile passes an image
bw4sz c43dcc2
edge case for image path
bw4sz fd19e07
batch results error
bw4sz 2c8d59e
fold then pad, not pad then fold
bw4sz 4aac4e9
crop result is wrong indent
bw4sz abc2221
skip if there is an empty image within a list
bw4sz a957fd1
take validation metrics out of epoch interval
bw4sz c8e2119
use predict batch instead of predict dataloader
bw4sz 64f1d34
postprocess dataloader
bw4sz 1f93d4e
move to batch on eval
bw4sz e54f400
specifically invoke eval mode
bw4sz 509ec5e
style edits
bw4sz b4757fe
add batch size arg to predict dataloader, related to https://github.c…
bw4sz 0150519
went back to passing path to val_step instead of a seperate loader
bw4sz 0f68ecb
add paths to tests for train boxdataset
bw4sz ea5672f
add a recreate model argument from the cropmodel
bw4sz 2b0d480
reverted model behavior to allow blank model if desired
bw4sz 90ac4de
style update
bw4sz 768321a
don't return model on self.create_model
bw4sz f50384e
typo in cropmodel test
bw4sz 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
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.
Uh oh!
There was an error while loading. Please reload this page.