Michael Baudin

Results 190 comments of Michael Baudin

I was not able to move the new classes to `experimental`. This was done in 29d054f59f77978b93f739d942d151d8c056c94c but it makes the unit test `t_LinearModelValidation_std.py` fail because the second input argument of...

@jschueller : I have an error with some unused variable: ``` lib/src/Uncertainty/Algorithm/MetaModel/FunctionalChaos/FunctionalChaosValidation.cxx:147:21: error: unused variable ‘indicesTrain’ [-Werror=unused-variable] 147 | const Indices indicesTrain(splitter.generate(indicesTest)); ``` Actually, the output of `splitter.generate()` is not...

I have an error that I cannot understand. Everything is fine before the LinearModelValidation is in `experimental`. After I moved the `LinearModelValidation` class to `experimental` in 8d94f0f9590e9c9ce0893f4c1e9a02f7a3c3c1ac, I got this...

You mean [CMakeLists.txt](https://github.com/openturns/openturns/blob/69816dc8b5c3319666dc917f1fa0b42e55195ed0/python/src/CMakeLists.txt#L875). I will try this.

@regislebrun : This PR seems to be ready from my point of view. Can you please review it?

> Can you revert [29d054f](https://github.com/openturns/openturns/commit/29d054f59f77978b93f739d942d151d8c056c94c) ? I am not able to cherry-pick this commit > > Will try to understand the issue... Thank you very much for the help. This...

> I don't understand what's happening... Any thoughts @jschueller ? > > If the `LinearModelValidation` and `FunctionalChaosValidation` are put in the `experimental` module, `LinearModelValidation` fails to run while setting arguments...

@jschueller : We are stuck here on the experimental issue. Do you have any idea?

> which issue ? If the LinearModelValidation and FunctionalChaosValidation are put in the experimental module, LinearModelValidation fails to run while setting arguments to ctor (otexp.LinearModelValidation() is ok)

> the modules of the argument classes probably need to be added to experimentail_module.i (before LinearModelValidation inclusion) Here is my analysis: - I need to import LinearModelResult, but this is...