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

Remove OpenGL calls from non-visual functions #11

Open
ocarre opened this issue Aug 28, 2017 · 3 comments
Open

Remove OpenGL calls from non-visual functions #11

ocarre opened this issue Aug 28, 2017 · 3 comments
Labels

Comments

@ocarre
Copy link

ocarre commented Aug 28, 2017

Many calls to OpenGL are done during component initialization, requiring a valid OpenGL context even if we are not drawing anything and crashing if there is none.

@ocarre ocarre added the cleanup label Aug 28, 2017
@maxime-tournier
Copy link

maxime-tournier commented Aug 28, 2017

Do we need an initGL api ? Or should we handle that internally on first draw ?

void Component::draw() { 
  if(!initGL) { 
     // ...
     initGL = true;
  }
  // ... 
}

@matthieu-nesme
Copy link
Member

Les VisualModels ont déjà une fonction initVisualpour ça.

@ocarre tu parles de composants visuels, ou de composant lambda qui ont un rendu de debug?
Pourrais-tu commencer par lister les composants problématiques ici ?

@ocarre
Copy link
Author

ocarre commented Sep 19, 2017

De composants qui crashent à l'init ou à l'update s'il n'y a pas de contexte opengl. Ce qui est dans les fonctions ***visual n'est en effet pas gênant. Oui il faudrait faire un listing, après faut les trouver les classes concernées ... lancer des scènes sans contextes opengl et sans visu et voir lesquels crashent -_- Je crois qu'il y en a pas mal dans le plugin image.

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

No branches or pull requests

3 participants