Skip to content

DNN: update network dump#16516

Merged
opencv-pushbot merged 2 commits intoopencv:3.4from
alalek:dnn_dump_network
Feb 11, 2020
Merged

DNN: update network dump#16516
opencv-pushbot merged 2 commits intoopencv:3.4from
alalek:dnn_dump_network

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Feb 5, 2020

  • don't require .setInput() before .dump()
  • dump networks before and after .setUpNet() through parameter
  • coding style changes in .dump() (tab => \t, indentation, "const").
force_builders=Custom,Custom Win,Custom Mac
build_image:Custom=ubuntu-openvino-2019r3.0:16.04
build_image:Custom Win=openvino-2019r3.0
build_image:Custom Mac=openvino-2019r3.0

test_modules:Custom=dnn,python2,python3,java
test_modules:Custom Win=dnn,python2,python3,java
test_modules:Custom Mac=dnn,python2,python3,java

buildworker:Custom=linux-1
# disabled due high memory usage: test_opencl:Custom=ON
test_opencl:Custom=OFF
test_bigdata:Custom=1
test_filter:Custom=*

allow_multiple_commits=1

Copy link
Copy Markdown
Member

@dkurt dkurt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants