Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Checking pass #14

Open
maxime-tournier opened this issue Aug 28, 2017 · 2 comments
Open

Checking pass #14

maxime-tournier opened this issue Aug 28, 2017 · 2 comments

Comments

@maxime-tournier
Copy link

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
Copy link
Member

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
Copy link
Author

maxime-tournier commented Sep 18, 2017

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants