Skip to content

add testdata for DataAugmentation with different sizes#780

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
zhaoyue-zephyrus:data_aug_with_different_size
Aug 26, 2020
Merged

add testdata for DataAugmentation with different sizes#780
opencv-pushbot merged 1 commit intoopencv:3.4from
zhaoyue-zephyrus:data_aug_with_different_size

Conversation

@zhaoyue-zephyrus
Copy link
Copy Markdown
Contributor

No description provided.

@l-bat
Copy link
Copy Markdown
Contributor

l-bat commented Jul 31, 2020

Please attach prototxt for tests

@l-bat
Copy link
Copy Markdown
Contributor

l-bat commented Aug 3, 2020

Please attach caffemodel for tests

@zhaoyue-zephyrus
Copy link
Copy Markdown
Contributor Author

Oops, caffemodel are ignored by unless using -f. Now fixed.

@l-bat
Copy link
Copy Markdown
Contributor

l-bat commented Aug 11, 2020

Please look at this warning https://pullrequest.opencv.org/buildbot/builders/precommit_docs/builds/26251/steps/patch%20size%20opencv_extra/logs/stdio
What is the main difference between data_augmentation_2x1 and data_augmentation_8x6 models? Can we only add data_augmentation_2x1 model to avoid exceeding the memory limit?

@zhaoyue-zephyrus
Copy link
Copy Markdown
Contributor Author

I use 2x1 and 8x6 to test that DataAug works for both enlarging (2x1 => 4x3) and shrinking (8x6 => 4x3). If I understand correctly, both model should have only hundreds of bytes?

@l-bat
Copy link
Copy Markdown
Contributor

l-bat commented Aug 12, 2020

I'm sorry if I gave you the wrong idea. Please rebase your PR to 3.4 branch.

@zhaoyue-zephyrus zhaoyue-zephyrus force-pushed the data_aug_with_different_size branch from aa06c07 to daa3f23 Compare August 12, 2020 09:30
@zhaoyue-zephyrus zhaoyue-zephyrus changed the base branch from master to 3.4 August 12, 2020 09:30
@opencv-pushbot opencv-pushbot merged commit 79d5090 into opencv:3.4 Aug 26, 2020
@alalek alalek mentioned this pull request Aug 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants