aremazeilles / beat_routine

Performance Indicators developed by BEAT consortium for the Eurobench project
0 stars 2 forks source link

CI process fails: check why #46

Closed aremazeilles closed 3 years ago

aremazeilles commented 3 years ago

The CI process has failed. I assume you should have received an email when you made your commit.

Opening the CI process failing, we can go and check the reason (click on the red cross, and go to details) The page brings you at the end, but it is better to go at the begining of the stage that fails: test the docker image

In this step, we launch your program (in docker) using the command placed in the test plan.

The first error is observable there

parse error near line 53 of file /home/octave/beat/Functions/posturographic_routine_unperturbed.m

  syntax error

>>>   a=sqrt(z)*D(1,1));

Effectively, you have an extra ) on the line. If you remove one, this trial will not fail.

We continue, skip the similar errors, and get to an error with the protocol5

First differing element 3:
'value: [0.000000, 0.000000, 0.000000, 0.000000, 0.000000[29 chars]0]\n'
'value: [0.000002, 0.000002, 0.000002, 0.000002, 0.000002[29 chars]2]\n'

This is related to the pi_eap.yaml file. In the reference output, we effectively find: 0.000002, but the generated file contains 0.000000. If the 0.00000 is the good value, then you should update the reference output. If not youshould check why the error happens. It seems to be the case also for the protocol_6.

Do you think you can handle this?

If the CI process fails something is wrong. This may be minor issue, but the verification does not distinguish minor and major deviation: no deviation allowed :).

When you push code and the cI process is failing, you should check where is the issue, and/or ask me support to handle it (but perhaps you did, and I forgot to answer...)

aremazeilles commented 3 years ago

Fix with #47