-
Notifications
You must be signed in to change notification settings - Fork 18.7k
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
DenseNet feature pyramid computation #307
Conversation
…and its python iface
…ors (as Sergey suggested)
…ew data type is dict['feat'] = list of feature scales
-- note: this commit almost certainly breaks compilation of matcaffe. mwm
…caffe output in matlab wrapper. mwm fix matlab vs octave compile stuff for matlab pyramid API
…-> caffe/src/stitch_pyramid
…to mean; corner regions still TODO
…rk; other minor changes. -- fill corners of padding with interpolation of edge padding (which is in turn already an interpolation from the image edge to the imagenet mean). -- add .PHONY stitch target to top level makefile for building just libStitchPyramid -- minor fix to testing makefile: add a -L. to src/stitch_pyramid/build/Makefile -- update matcaffe.cpp comment with current mkoctfile-based build command (can be used to test building matcaffe under octave) -- condense str() in featpyramid_common.hpp (str() is for debugging printfs) -- add a copy of str() in JPEDPyramid.cpp (FIXME: use some common copy?) mwm
…ject*) casts (to fix compile errors for some build envs). note also that there is a new SHARED_LDFLAGS makefile var that can be set to include -Wl,--no-undefined (for gcc) to avoid accidentally linking an .so missing some of its dependencies. but, since people use other compilers, it's not enabled by default, and only the stitch library build line uses the macro at this point. mwm
…port note that matcaffe was broken (would compile, but would have incorrect behavior) when feat_min{Width,Height} were added these since the function that should have been taking them has 5 int args, all with defaults, and new args were added in middle of the arg list.
… matlab. also, commenting out InitGoogleLogging() in matcaffe until we decide how to avoid multiple calls to InitGoogleLogging
…ong. at a minimum, maybe we should have a way to disable trying to init logging (i.e. as a param to caffe('init')) as a workaround for the multiple-library usage case?
…d' for building pycaffe
…on/license note there too.
Added some DenseNet API documentation. This will probably percolate from this top-level README.md to the caffe.berkeleyvision.org gh-pages.
Hey Matt, I'm excited to see this code go public! However, you should close this PR and open one against the |
duh. will do. new PR is #308 On Tue, Apr 8, 2014 at 1:15 PM, Evan Shelhamer notifications@github.comwrote:
|
Hi, Matt, Could you provide the link for downloading the DenseNet code? I was Thanks~ Milton 2014-04-09 3:33 GMT+08:00 moskewcz notifications@github.com:
|
|
DenseNet PR current state / TODOs (for discussion)
explanatory tech report:
DenseNet: Implementing Efficient ConvNet Descriptor Pyramids
Forrest Iandola, Matt Moskewicz, Sergey Karayev, Ross Girshick, Trevor Darrell, and Kurt Keutzer
additional integration notes (see main notes below inlined from the DENSENET_MERGE_TODO file)
inlined from: DENSENET_MERGE_TODO
list of the issues blocking the merge of the DenseNet feature branch: