Checking pass - Githubissues

Anatoscope / sofa

Real-time multi-physics simulation with an emphasis on medical simulation.

https://www.sofa-framework.org

2 stars 1 forks source link

Checking pass #14

Open maxime-tournier opened 7 years ago

maxime-tournier commented 7 years ago

Currently we have lots of ad-hoc checking happening inside components.

We could instead add a check method that

checks internal consistency
validates inputs for mappings
resizes outputs for mappings?

and have a dedicated CheckVisitor sent at relevant times.

Also: use exceptions for error handling !

Suggested labels:

matthieu-nesme commented 7 years ago

Ca a déjà été discuté et acté pour sofa public. Cette fonction pourrait aussi être appelée dans runSofa quand on change quelque chose à la main depuis la gui. En tout cas les verifs ne seraient pas effectuées tout le temps, pour ne pas perdre de temps quand ce n'est pas nécessaire. A faire dans sofa-public pour ne pas diverger sur ce point.

maxime-tournier commented 7 years ago

Ok alors il faudrait:

une methode BaseObject::check qui lance une classe dérivée de std::runtime_error en cas d'erreur (genre sofa::error)
test + gestion des exceptions dans les visiteurs (init, animate, etc)

Note que vu qu'on peut a priori tout faire peter d'un pas de temps sur l'autre avec python on est obligés de faire les tests en debut de pas de temps.