Re: [eigen] Help to compile/run unit tests for next 3.3.5 release

Looks like it worked now, excellent! It took 2-3 hours or something like that now and it is uploaded.

I'll run 15.6.0 Preview 1.0 now too.

Let me know if you're interested in running it with any other settings/flags, as long as it's VS2017 or 2017 Preview I'm happy to run.

Best wishes,

Patrik

On 15 December 2017 at 22:31, Gael Guennebaud <gael.guennebaud@xxxxxxxxx> wrote:

On Fri, Dec 15, 2017 at 6:08 PM, Patrik Huber <patrikhuber@xxxxxxxxx> wrote:
Hi Gael,

I've started builds on VS2017 15.5.1 and VS2017 15.6.0 Preview 1.0 with and without -DEIGEN_TEST_CXX11=ON (I thought that might be useful) between 12 and 2pm, one with -j 9 and the other one without. Both have been running since and haven't completed yet.
One is currently at "Size: 1299K", the other one at "2050K". I read on the wiki that a full rebuild can take up to three hours, but particularly with -j 9 and a good machine (SSD, recent quad-core i7), I am surprised. Is this to be expected?

Thanks a lot for starting the builds. I run it with MSVC 2012 in a VM without multithreading and it completed in 2 to 3 hours max. The build/test/Debug folder is about 6GB. So:
1 - make sure you've updated to 3.3 (the devel branch is more involved, because there are more tests!)
2 - To speed up builds you can add /DEIGEN_STRONG_INLINE=inline in CMAKE_CXX_FLAGS
3 - "ctest -j 9" won't speedup the builds, but only the run of the tests.

So for faster parallel builds (4 processus):

cmake . -DCMAKE_CXX_FLAGS="/MP4 /DEIGEN_STRONG_INLINE=inline"

I've essentially followed your instructions and then

cmake .. -G "Visual Studio 15 2017 Win64"
ctest -C Release -D Experimental -j 9

>> there is small issue, the last line should be:
>> ctest -D Experimental

So that means no "-C Release"? What will that run?

all tests will be run instead of the ones in eigen/test only (in theory, here none were run, which is why I recommended to remove this option).

In case of problems, you can rerun the tests without rebuilding binaries that are already up-to-date with:

ctest -D Experimental --build-noclean -VV

-VV is useful to see what's going on...

thanks again.

I'm happy to let them run & finish or restart them with modified commands, just let me know.

Kind regards,

Patrik

--
Dr. Patrik Huber
Centre for Vision, Speech and Signal Processing
University of Surrey
Guildford, Surrey GU2 7XH
United Kingdom

Web: www.patrikhuber.ch
Mobile: +44 (0)7482 633 934

On 15 December 2017 at 16:59, Gael Guennebaud <gael.guennebaud@xxxxxxxxx> wrote:

On Fri, Dec 15, 2017 at 9:54 AM, Gael Guennebaud <gael.guennebaud@xxxxxxxxx> wrote:
[...]
ctest -C Release -D Experimental

there is small issue, the last line should be:

ctest -D Experimental

For the record, the build is green for msvc 11 (2012), it would still be welcome to get feedback for MSVC 2015 and 2017.

gael

--
Dr. Patrik Huber
Centre for Vision, Speech and Signal Processing
University of Surrey
Guildford, Surrey GU2 7XH
United Kingdom

Web: www.patrikhuber.ch
Mobile: +44 (0)7482 633 934

Dr.. Patrik Huber
Centre for Vision, Speech and Signal Processing
University of Surrey
Guildford, Surrey GU2 7XH
United Kingdom

Web: www.patrikhuber.ch
Mobile: +44 (0)7482 633 934