Date

Sep 12, 2014

Attendees

 

John Clyne, Alan

Notes

DataMgr and renderer

Discussed Alan's DataStatus API document DataStatusAPI.pdf and made decisions about which methods would be most appropriate for the DataMgr, and which should remain in the DataStatus class (or elsewhere)

Agreed to change Renderer API to pass DataMgr and RendererParams as arguments to Renderer::PaintGL(). This would do away with treating these as global variables and would alleviate need for special handling of Undo/Redo: Undo and Redo would be possible by simply passing in the new Params object as a parameter.

Discussed the need for additional Param classes Set and Get methods that would allow for dirty bit flags. These will be implemented and may be used by the Flow renderer

Agreed to add return status codes to the Renderer's PaintGL and InitializeGL methods. Derived classes would be responsible for returning an error status and logging an error message to MyBase. We will also look at adding the protected, pure virtual methods _paintGL() and _initializeGL() to the renderer. Derived classes would implement these and the Renderer paint and initialize methods could be implemented:

int Renderer::paintGL() {

  int rc = _paintGL()

  if (rc<0) return(rc)

  return(Renderer::GetOpenGLStatus())

}

John will add a GetOpenGLStatus() method to the renderer to check for OpenGL errors()

GUI

We've not previously had any discussion on architectural or layout changes desired for the GUI. Scott's proposed new tab layout looks promising, but needs to be explored further. The internal architecture also needs to be explored and desired improvements identified.

Action Items

ItemLead
Add missing DataMgr methods identified in DataStatus documentJohn
Revise DataStatus class in response to above DataMgr changesAlan
Refactor Renderer to accept DataMgr and Params as arguments, add return status to init and paint methodsAlan
Assess state of GUI internals and propose needed changesAlan
Evaluate possibility of migrating to Qt 5Alan
Implement OpenGL status method on rendererJohn