Skip to content

Comments

Compile errors in boost-eigen branch#47

Merged
jeffdonahue merged 2 commits intoBVLC:boost-eigenfrom
alito:compileerrorsboosteigenkloudkl
Jan 22, 2014
Merged

Compile errors in boost-eigen branch#47
jeffdonahue merged 2 commits intoBVLC:boost-eigenfrom
alito:compileerrorsboosteigenkloudkl

Conversation

@alito
Copy link

@alito alito commented Jan 22, 2014

The boost-eigen branch does not compile for me. This is after the merge from kloudkl (8d894f0) (I never tested it before that).

boost::math::nextafter should only be parametrised on one type (or so says gcc here, and what I read on the boost docs).

Also, the tests for test_random_number_generator reference mean_bound and sample_mean but they need to be referenced as this->mean_bound (again, that's just what my gcc thinks).

This branch contains those two fixes. After this, everything compiles and the tests run fine (except for some GPU tests)

@dnouri
Copy link

dnouri commented Jan 22, 2014

Sorry, it must have been that I tested an older version of kloudkl's branch when I confirmed it was working. :-(

I can anyway now confirm that merging alito's branch into boost-eigen allows me to compile and run caffe again. I've run all tests and two of them are failing:

  • "PoolingLayerTest/1.TestGPUGradientAve" errors with "pooling_layer.cu:309] Cuda kernel failed. Error: too many resources requested for launch"
  • "LRNLayerTest/0.TestGPUGradient" errors with "lrn_layer.cu:80] Cuda kernel failed. Error: too many resources requested for launch"

I haven't tested against the main branch, so can't say if these occur there as well.

@jeffdonahue
Copy link
Contributor

Thanks @alito! We're confirming this builds and passes with both Boost 1.46 and Boost 1.54. (kloudkl's version compiles with Boost 1.54, but not 1.46.)

jeffdonahue added a commit that referenced this pull request Jan 22, 2014
@jeffdonahue jeffdonahue merged commit 3122c8a into BVLC:boost-eigen Jan 22, 2014
@alito alito deleted the compileerrorsboosteigenkloudkl branch January 23, 2014 11:02
@kloudkl
Copy link
Contributor

kloudkl commented Jan 23, 2014

All the tests passed under the environment as follows.
gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3
Boost 1.54/1.55 installed from source
NVIDIA GTX 560 Ti with 1GB memory)
CUDA 5.5

Is it necessary to setup a continuous integration infrastructure like the Chromium build infrastructure to test various possible combinations?

Cysu added a commit to Cysu/caffe that referenced this pull request Feb 24, 2016
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.

4 participants