Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Redesign internal augmentation methods #451

Merged
merged 139 commits into from Oct 14, 2019
Merged

Redesign internal augmentation methods #451

merged 139 commits into from Oct 14, 2019

Conversation

aleju
Copy link
Owner

@aleju aleju commented Oct 4, 2019

No description provided.

@codecov-io
Copy link

codecov-io commented Oct 5, 2019 •
edited

Codecov Report

Merging #451 into master will decrease coverage by 0.23%.
The diff coverage is 97.22%.

@@            Coverage Diff             @@
##           master     #451      +/-   ##
==========================================
- Coverage   96.16%   95.93%   -0.23%     
==========================================
  Files          34       34              
  Lines       11092    11178      +86     
==========================================
+ Hits        10666    10723      +57     
- Misses        426      455      +29

@aleju
Add get_augmentable_names_to_augment() to batches
53c392d
@aleju
Add to_normalized_batch() to Batch
b0a59f2
@aleju
Add BatchInAugmentation
398877a
@aleju
Add to_batch_in_augmentation() to Batch and UnnormalizedBatch
64e1846
@aleju
Add docstring for get_augmentable_names_to_augment()
c2e6b78
@aleju
Make _maybe_deterministic_context() more flexible
a230845
@aleju
Add Augmenter._augment_batch() method
9b42e27
@aleju
Rename _maybe_deterministic_context() to *_ctx()
8eda26b
@aleju
Add activation checks and hooks to augment_batch()
440ff0e
@aleju
Change augment_heatmaps() to wrapper around augment_batch()
98971c5
@aleju
Change augment_segmentation_maps() to wrapper around augment_batch()
77d75d1
@aleju
Change augment_keypoints() to wrapper around augment_batch()
b0f41f5
@aleju
Change augment_bounding_boxes() to wrapper around augment_batch()
a60ecec
@aleju
Change augment_polygons() to wrapper around augment_batch()
02a837b
@aleju
Change augment_line_strings() to wrapper around augment_batch()
e25754b
@aleju
Fix image inverse-normalization sometimes failing if ndarray turned t…
e4c6a2a 
…o list
@aleju
Change augment_images() to wrapper around augment_batch()
1f8d566
@aleju
Add docstring for _augment_batch()
fc5c03d
@aleju
Change _augment_images() to default to noop
d9635c7
@aleju
Fix typo in image normalization error message
fbbf2c7
@aleju
Add get_augmentables() and get_augmentables_names()
8632362
@aleju
Add fill_from_batch_in_augmentation_() to batches
9f8ea38
@aleju
Add propagation context to BatchInAugmentation
5ee219e
@aleju
Refactor augment_batch() and _augment_batch()
39a017d
@aleju
Switch Sequential to new _augment_batch() interface
e112a5f
@aleju
Add nb_items to BatchInAugmentation
e9b4346
@aleju
Add subselection methods to BatchInAugmentation
c3a7677
@aleju
Switch SomeOf to augment_batch() interface
b92f064
@aleju
Switch Sometimes to augment_batch() interface
e46a1dd
@aleju
Add tests for batches module
5d9c6cc
@aleju
Use deepcopy_fast() in BIA.deepcopy()
8bc3a27
@aleju
Fix deprecation warnings in tests
3b833e2
@aleju
Fix docstrings
63052f0
@aleju
Add tests for new kpsoi methods
2f28996
@aleju
Fix fill_from_xy_array_() not accepting lists
32e939d
@aleju
Fix docstrings not listing all valid datatypes
1a439f7
@aleju
Add tests for new bbsoi methods
80839c9
@aleju
Add tests for new lsoi methods
954dbd8
@aleju
Add tests for new psoi methods
198e36d
@aleju
Add BB.from_point_soup()
8ea92ac
@aleju
Change BBSOI.from_xyxy_array() to also accept (N, 2, 2)
e30607a
@aleju
Improve import statement
bb42e7e
@aleju
Add BBSOI.from_point_soups()
10f12ff
@aleju
Switch BB aug from 2 to 4 corners by default
353c9c1
@aleju
Add and adapt tests related to BBs
4a6c91c
@aleju
Synchronize changelog
9e80eaa
@aleju
Fix recover_psois_() causing shape errors
b9cdeb1
@aleju
Move legacy auto-deterministic out of augment_batch()
234b572
@aleju
Remove Augmenter._is_activated_with_hooks()
e8a94a9
@aleju
Remove Augmenter._augment_heatmaps_as_images()
c6df364
@aleju
Remove unused import
baa0437
@aleju
Add notes and warnings to docstrings
ea8988f
@aleju aleju merged commit 2d3fdb7 into master Oct 14, 2019
@aleju aleju deleted the single_step_augment branch October 14, 2019 10:37
aleju added a commit that referenced this pull request Nov 3, 2019
@aleju
Fix Affine BB aug using only 2 corner points
0ee793d 
This fixes a bug where the bounding box augmentation
in Affine uses only two corner cordinates (top-left,
bottom-right) instead of all four, leading to malformed
output bounding boxes during rotation and shear.

This bug was likely introduced by #451 (2d3fdb7,
single_step_augment).
@aleju aleju mentioned this pull request Nov 3, 2019
Merged
aleju added a commit that referenced this pull request Nov 4, 2019
@aleju
Fix Affine BB aug using only 2 corner points
8ac38b3 
This fixes a bug where the bounding box augmentation
in Affine uses only two corner cordinates (top-left,
bottom-right) instead of all four, leading to malformed
output bounding boxes during rotation and shear.

This bug was likely introduced by #451 (2d3fdb7,
single_step_augment).
@aleju aleju mentioned this pull request Jan 12, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@aleju @codecov-io