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.

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

Tencent/ncnn 2020 02 07 #5

Open
wants to merge 223 commits into
base: master
Choose a base branch
from
Open

Conversation

fantes
Copy link

@fantes fantes commented Feb 7, 2020

(+ our small changes)

nihui and others added 30 commits September 26, 2019 14:43
* Fixed lots of compile warnings

* refine the unused warning change
nihui and others added 28 commits January 17, 2020 22:56
baseclass Pipeline use virtual destruction
* skip compile caffe tools

* add convolution_int8 requant test

* fix

* test int8requant

* revert code

* clean code

* fix CI error

* resolve review advices

* Update convolution_arm.cpp

Co-authored-by: nihui <[email protected]>
* 1. for cross platform, use cv::CommandLineParser instead of getopt.h
2. use cv::glob to search folder instead of dirent.h
3. fix some other warnings of function from cstdlib
4. add some const and some auto

* 1. fix some other warnings of function from cstdlib
2. add some const and some auto
3. i forgot...

* switch quantize tools default on

* check OpenCV first, if not found, disable quantize tools

* add _CRT_SECURE_NO_DEPRECATE for msvc and remove *_s functions

* add _CRT_SECURE_NO_DEPRECATE for msvc and remove *_s functions

* keep the one line style block

* folding bracket

* dynamic_cast -> static_cast
* revise more understandable help info

* try update ubuntu system before apt-get install

* remove upgrade
…t#1524)

* fix type conversion warning

* fix MSVC _CRT_SECURE_NO_DEPRECATE warning

* type conversion: missing one

* fix  MSVC _CRT_SECURE_NO_DEPRECATE warning and type warning

* reformat

* convert google::protobuf::int64 to size_t to avoid type impl between msvc and gcc gap

* remove C++11 auto

* tab -> space
@beniz
Copy link

beniz commented Feb 19, 2020

Thanks, tested with DD ?

@fantes
Copy link
Author

fantes commented Feb 24, 2020

yes

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.