[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [MoM] FAST test suite issues



Hi Shayan,

I've now tested what my desktop has build yesterday. :-)

On Mon, Sep 02, 2019 at 12:57:55AM +0100, Shayan Doust wrote:
> 
> I also see no reason why the test suite will not work now. As far as the
> binary executes for me (until the opencl limitation), I see no asset-
> related assertion or error. Curious to see if we get a success with testing.

I was running `sh debian/tests/run-unit-test 2>&1 | tee > fast_test.log`
and attached this log here.  The end of this log is

...
INFO: QApp already exists..
INFO: Device manager initialize..
-------------------------------------------------------------------------------
Pipeline C
-------------------------------------------------------------------------------
/build/fast-3.0.0~rc3+ds/source/FAST/Tests/Benchmarks.cpp:167
...............................................................................

/build/fast-3.0.0~rc3+ds/source/FAST/Tests/Benchmarks.cpp:167: FAILED:
due to unexpected exception with message:
  clGetPlatformIDs

===============================================================================
test cases: 221 |  32 passed | 189 failed
assertions: 385 | 196 passed | 189 failed


So at least not everything you did is failing. ;-)

I also tried to simply call the server:

$ LC_ALL=C /usr/lib/fast/OpenIGTLinkServer
WARNING: Unable to open the configuration file /usr/fast_configuration.txt. Using defaults instead.
terminate called after throwing an instance of 'fast::Exception'
  what():  Unable to open directory /usr/share/fonts/truetype/ubuntu
Abgebrochen

(Sorry for the German locale - I've set LC_ALL=C but it did not helped)

After installing fonts-ubuntu I've got a graphical interface.  So
I just added fonts-ubuntu to Depends of libfast0 (may be it would
be sufficient to add it to fast binary package but I think it does
not harm inside the library) as well as Build-Depends (I'm not fully
sure whether this is really needed but might not harm as well).

Unfortunately the installation of fonts-ubuntu does not change much
in the test suite.  I repeated it and here is the diff of the logs:

--- fast_test.log       2019-09-05 07:25:40.037252840 +0200
+++ fast_test_2.log     2019-09-05 07:40:21.076277590 +0200
@@ -2,6 +2,10 @@
 Invoking testFAST
 INFO: Creating new QApp
 WARNING: Unable to open the configuration file /usr/fast_configuration.txt. Using defaults instead.
+INFO: Creating new GL context for computation thread
+WARNING: Your system uses comma as decimal point.
+WARNING: This will now be changed to dot to avoid any comma related bugs.
+INFO: Device manager initialize..
 
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 testFAST is a Catch v1.5.9 host application.
@@ -15,12 +19,9 @@
 
 /build/fast-3.0.0~rc3+ds/source/FAST/Algorithms/AirwaySegmentation/airwaySegmentationTests.cpp:49: FAILED:
 due to unexpected exception with message:
-  Unable to open directory /usr/share/fonts/truetype/ubuntu
+  clGetPlatformIDs
 
 INFO: QApp already exists..
-INFO: Creating new GL context for computation thread
-WARNING: Your system uses comma as decimal point.
-WARNING: This will now be changed to dot to avoid any comma related bugs.
 INFO: Device manager initialize..
 -------------------------------------------------------------------------------
 Block matching 2D


Its basically that some locale related things (dot -> comma) are
showing up at different places and that the exception about the
fonts changed.  I'm a bit worried about

   fast_configuration.txt

Configuration files should be in /etc/fast/fast_configuration.txt.  You
should probably patch the code to seek there and we should also provide
some configuration file there.  However, I doubt that those test suite
errors are occuring due to missing configuration.

That's all I can say for now.

Thanks again for your work

    Andreas.

Attachment: fast_test.log.gz
Description: application/gzip


Reply to: