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

Shugeo suppression overlaps #9

Merged
merged 27 commits into from Oct 30, 2019
Merged

Shugeo suppression overlaps #9

merged 27 commits into from Oct 30, 2019

Conversation

shugeo
Copy link

@shugeo shugeo commented Oct 22, 2019

What changes were proposed in this pull request?

Implementation of non_max_suppression_overlaps op was added. Implementation of non_max_suppression op was refactored to conform with TF v.2.0

How was this patch tested?

Proper tests were added.

Quick checklist

The following checklist helps ensure your PR is complete:

  • Eclipse Contributor Agreement signed, and signed commits - see IP Requirements page for details
  • Reviewed the Contributing Guidelines and followed the steps within.
  • Created tests for any significant new code additions.
  • Relevant tests for your changes are passing.

@shugeo
Added non_max_suppression_overlaps op and tests.
670a9fe
@shugeo
Merge branch 'master' of https://github.com/KonduitAI/deeplearning4j …
f0a5b4b 
…into shugeo_suppression_overlaps
@shugeo
Refactored implementation of non_max_suppression_overlaps.
3e8d8f1
@shugeo
Refactoring of implementation of non_max_suppression_overlaps op.
8ea4d2a
@shugeo
Refactoring of implementation of non_max_suppression op.
874bed6
@shugeo
Fixed portion error.
4d9b371
@shugeo
Added cuda frontends for image suppression ops.
da7aa39
@shugeo
Eliminated crash with cuda arch on image.non_max_suppression_overlaps…
eaa1f61 
… op.
@shugeo
Improved implementation of image_suppression helper for cpu platform.
59d0928
@shugeo
The generic approach of non_max_suppression_overlaps op helper with c…
c65dd4c 
…uda platform.
@shugeo
Working cuda implementation of helper non_max_suppression_overlaps op.
c6ab768
@shugeo
Eliminated waste comments.
cc6c4e3
@shugeo
Improved implementations for both platforms
cbf2725
@shugeo
Refactored cuda implementation of image.non_max_suppression_overlaps …
9fa183e 
…op helper.
@shugeo
Improved cuda implementation of non_max_suppression op helper.
54cb8ee
@shugeo
Refactored cuda implementation of image.non_max_suppression_overlaps …
fbf8efc 
…op helper.
@shugeo
Improved cuda implementation of image.non_max_suppression_overlaps op…
0b4f094 
… helper.
@shugeo
Added modifications into cuda implementation for image suppression ov…
c0d2185 
…erlaps op.
@shugeo
Correct queue emulation with cuda implementation of non_max_suppressi…
63f9fdb 
…on_overlaps op.
@shugeo
Prefinal stage of cuda implementation of non_max_suppression_overlaps.
8c6999d
@shugeo
Worked cuda implementation of non_max_suppresion_overlaps helper.
0801f4c
@shugeo
Fixed return to proper thread.
7e7261e
@shugeo
Improvements for cuda implementation of image.non_max_suppression_ove…
2570a9e 
…rlaps op helper.
raver119 pushed a commit that referenced this pull request Oct 26, 2019
@alexanderst
Merge pull request #9 from KonduitAI/asto_nd4s_indexing
fa8105b 
[WIP] Indexing for INDArray in nd4s
@shugeo shugeo changed the title [WIP] Shugeo suppression overlaps Shugeo suppression overlaps Oct 30, 2019
@shugeo shugeo merged commit 95f7ad7 into master Oct 30, 2019
@shugeo shugeo deleted the shugeo_suppression_overlaps branch October 30, 2019 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@raver119 raver119 raver119 approved these changes

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
@shugeo @raver119