-
Notifications
You must be signed in to change notification settings - Fork 19
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
Enable patchwise training and prediction #135
Open
davidwilby
wants to merge
153
commits into
alan-turing-institute:main
Choose a base branch
from
davidwilby:patchwise_train
base: main
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.
+1,895
−42
Open
Changes from 22 commits
Commits
Show all changes
153 commits
Select commit
Hold shift + click to select a range
131c434
stach changes
nilsleh 3342b96
draft
nilsleh b7cf3fa
draft
nilsleh 70f3783
merge main
nilsleh 379e3b2
wrong merge
nilsleh 85cd34b
incorporate some of the feedback
nilsleh be8fffd
run black
nilsleh 3415377
merge main
nilsleh 39dd15b
merge main
nilsleh 876970e
layout code
nilsleh d1cb338
change __call__
nilsleh 218f791
revert
nilsleh 37fe771
type annotation
nilsleh fb20ccc
patch_size sampling test
nilsleh 5bda80b
patchwise test trainer
nilsleh c276844
gridded window patching
fde7e02
adding sliding window patching function
195a923
loader with bboxes
nilsleh 824df24
loader with boxes
nilsleh e6e1ae8
Altering kwargs to enable for-loop and change sliding function
e75d022
Merge branch 'patchwise_train' into msjr/patching
nilsleh bae0855
move logic to call
nilsleh a090d34
Merge branch 'main' into patchwise_train
nilsleh 797f48e
Merge branch 'patchwise_train' into msjr/patching
nilsleh 5291ec3
Merge pull request #1 from nilsleh/msjr/patching
nilsleh 7b09119
typo
nilsleh 282c2be
notebook with patchwise train
nilsleh dfa386d
refining stride to avoid error
8d46653
inference patching
acbad8b
predict_patches
3e2994e
patchwise predictions during inference and stitching
765849d
fix typo
abc9a9d
merge main
nilsleh 7f8ef93
new cropped stitching
847a47c
clipped patchwise predictions, single date
f93fc39
correct minor errors/typos
07256f0
Merge pull request #2 from nilsleh/msjr-test_patching
nilsleh d8af314
use TODO to be uniform
davidwilby f3b7f12
use "stride" as in taskloader
davidwilby 5a1766b
resolve unnormalised coordinate names
84d9944
Handle absent bbox and task as non-iterable
davidwilby aab6f1e
resolve unnormalised coordinate names
bda7176
use dict format for isel for variable coordinate names
davidwilby 55bf86f
add basic test for patchwise prediction
davidwilby 323ab46
handle patch_size and stride as floats or tuples in task loader and p…
davidwilby 09befb3
test parameter handling and sizes in patchwise prediction
davidwilby c36455b
remove resolved TODO
davidwilby 0cf143d
check patch_size and stride values in predict_patch and test
davidwilby 8da48c1
test inference
nilsleh 7cf556e
correct typo
ca6d001
resolve conflicts
bc862df
fix stride & patch checking
davidwilby 61dc88e
revert previous commit
nilsleh f2bd5bb
fix patchwise training tests
davidwilby 601102e
add actual training step to test_sliding_window_training
davidwilby 64773fc
try to make printing work for task objects with bbox attribute
davidwilby 69f0ac6
run black
davidwilby f5e4a8a
re-add missing code from task loader
davidwilby 8aac7b6
Merge pull request #5 from nilsleh/dw/add_missing_loader_code
nilsleh c7a6172
Merge pull request #3 from nilsleh/msjr-test_patching
nilsleh 5e29031
Commit to allow patching irrespective of whether x1 and x2 are ascend…
294cc47
changes to loader.py to ensure all patched tasks run left to right an…
4e136e3
Commit to make model agnostic to coord direction
6dd9b3a
Resolve conflicts primarily due to use of unnorm_name and orig_name
529e8c8
use more informative error message for predict_patch
davidwilby f37e28c
Merge branch 'patchwise_train' into dw/refactor_predict
davidwilby 0344c2a
fix use of stride_size
davidwilby 840838d
move patchwise parameter test to test_task_loader
davidwilby ceeb8ca
fix patch_size and stride for sliding window tests
davidwilby 5fc1fe3
remove test as moved to test_task_loader
davidwilby 1f434cc
check input parameters in task loader
davidwilby 96edce8
For patchwise prediction, get patch_size and stride directly from task
davidwilby a705549
Merge pull request #6 from nilsleh/msjr-test_patching
nilsleh f5c015b
Merge branch 'patchwise_train' into dw/patch_size_from_task_for_predict
davidwilby c14d5d1
Merge pull request #7 from nilsleh/dw/patch_size_from_task_for_predict
nilsleh 18f2e5a
raise errors instead of assert
davidwilby 47d0998
use warning for stride > patch size
davidwilby df0533b
remove comment
davidwilby f7d57e9
raise error for stride > patch_size in prediction
davidwilby fed3940
alter paramaters for test
davidwilby b3a6dab
raise error for more than one date in predict_patch
davidwilby c8a38f2
black
davidwilby 52a0cb3
Merge branch 'main' into dw/merge_main
davidwilby 2672fec
Merge branch 'patchwise_train' into dw/refactor_predict
davidwilby e10d645
fix getting and checking of patch_size and stride
davidwilby f6f843d
fix docstrings and defaults
davidwilby 2e5c6a8
reinstate orig_name patch clip slicing
davidwilby e4d5567
Merge pull request #8 from nilsleh/dw/merge_main
nilsleh e57f065
Merge branch 'patchwise_train' into dw/refactor_predict
davidwilby 51d8c05
use hypothesis to expand on patchwise predict testing
davidwilby 79afa12
Merge pull request #4 from nilsleh/dw/refactor_predict
davidwilby d6500f7
account for warnings/errors in patchwise task loader
davidwilby 7c47357
allow longer test runs
davidwilby 274902a
use patch size which relates to the normalised size
davidwilby 60c7a14
alter docstring to reflect function
davidwilby cdbd73a
attempt fix for compute_x1x2_direction
davidwilby f1e3dfe
Merge branch 'dw/compute_coord_direction' into dw/fix_regression
davidwilby 6f0e2e6
Merge pull request #5 from davidwilby/dw/fix_regression
davidwilby c0cd17e
address montonic and prediction size issues
d30e687
move patchwise test out of class
davidwilby 7a100ee
Update deepsensor/model/model.py
MartinSJRogers 23733df
Merge pull request #8 from davidwilby/montonic_errors
MartinSJRogers 4a4276d
Move spatial slicing below gapfill sampling
2f0e2ba
Merge pull request #11 from davidwilby/gapfill_loop
MartinSJRogers e2488f3
Merge branch 'main' into patchwise_train
davidwilby 54ee611
Merge remote-tracking branch 'origin/patchwise_train' into patchwise_…
davidwilby 9b1f30d
lint patchwise code
davidwilby 812e056
Update patchwise training notebook with additional descriptive text
3a34ed3
Merge pull request #13 from davidwilby/patchwise_linting
davidwilby 4859f2d
rename notebook; use new tqdm notebook; other small tweaks to text
davidwilby 53f238f
add correct output and prediction plot
davidwilby 9e4254f
Merge branch 'patchwise_train' into update_notebook
davidwilby 3e68ee1
Merge pull request #14 from davidwilby/update_notebook
davidwilby f7d5422
use python 3.8 compatible type hints
davidwilby 527edff
rename predict_patch to predict_patchwise and fix references
davidwilby afac690
remove mention of contributing in error message
davidwilby 277cdc3
refactor overlap calculation
davidwilby 88ae024
use smaller test dataset
davidwilby 325de6d
update docstring for predict_patchwise
davidwilby 9d79b34
account for non-gridded data correctly
davidwilby 4c05b77
refactor to reduce duplication; reduce floating point errors
davidwilby 4b570ae
first attempt using merge
df88bc7
pass kwargs to predict; use data_processor attribute instead of arg
davidwilby 4e028ab
correct typo
davidwilby 657a42a
Replace combine by coords with method to infill blank prediction obje…
e68c01a
remove the +1 to prevent Nan lines forming
8b9a8ac
add some comments
davidwilby d620e88
Merge pull request #16 from davidwilby/refactor_sample_sliding
davidwilby 3c1c1c8
Update deepsensor/model/model.py
MartinSJRogers e601109
linting
davidwilby a86ce31
tweak comments
davidwilby c7a994e
re-enable size checking in test
davidwilby e5b580b
rename some variables for slightly improved readability; add typehints
davidwilby 91b83ce
Merge pull request #18 from davidwilby/replace_combineByCoords_with_m…
davidwilby 158b6dc
Merge pull request #17 from davidwilby/predict_args
davidwilby 747f7dd
reduce large comment block to easier to follow inline comments
davidwilby 4f5eead
remove unused hypothesis dependency
davidwilby 322766f
remove todo
davidwilby b4e9ff5
move coord direction calcuation to where it is needed
davidwilby 572d7ec
clean up markup
da2f68f
Reduce repitiion and place code to determine coordinate extent in one…
c2f0ffe
Create DeepSensor object straight after stitching
9a7e743
Slightly amend some mark up text
e857355
Editted text for get_coordinate_extent_method
358b884
Edit where time is defined in stitched prediction object
58e9076
Reduce for loops and keep predictions as deepsensor.prediction objects
9943e99
Update deepsensor/model/model.py
MartinSJRogers 53ee50f
Merge pull request #19 from davidwilby/simplify_stitching
davidwilby 6cf0a28
Update deepsensor/model/model.py
MartinSJRogers 1f0fb32
Merge pull request #20 from davidwilby/simplify_stitching_retain_pred…
davidwilby be883dc
lint
davidwilby b0459e8
use python 3.8 compatible typehint
davidwilby 9765787
correct type hint
davidwilby 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 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.
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.
I can imagine the
num_samples_per_date
kwarg name being confusing for users because it's not obvious it relates to patching. Does it make more sense to call itnum_patch_tasks
?