Skip to content

Rework Main Application from the ground up

Arnaud Billon requested to merge fix-ruler into master

This is a Draft of baseline changes in an effort to lidarview-based apps coherence, and applied in a near future to all LV-based apps.

Related to : lidarview-core!267 (merged)

Changelog

Rework MainWindow initialization

  • Conform to Paraview mainWindow inheritance
  • Sort verbatim PV / LV-Base common elements / App specifc elements
  • Keep Base PV elements in the same order as the PV mainWindows for ease of maintenance
  • Keep Base PV elements commented out if not used for ease of maintenance

Split pqLidarViewManager into LVCore basic features and LidarView-based app specific manager

  • Used Paraview static singleton inheritance pattern
  • Split associated PythonQt Decorators
  • Renamed to lqLidarViewManager and lqLidarCoreManager

Reworked Custom VCR Controller

  • Subclassed PV VCR controller for ease of maintenance
  • Removed renderView dependency / init order criticality
  • Removed useless cache variables

Reworked lqSensorWidget

  • Allow checking for hasLidarSource() and bool isInLiveSensorMode(), instrumental in checking for "Live Stream mode"
  • Added Signal lidarStreamModeChanged(bool); // Emitted when LidarStream are now present / absent.
  • Added additional warning in lqSensorListWidget::onSourceRemoved(pqPipelineSource *src)

Reowrked Applogic

  • Removed significant dead code portions
  • Removed Ruler related code superseded by reaction
  • Tweaked Grid Creation

Fixed UI Layout SettingRestoration

Shortened SOFTWARE_VEDOR from Kitware, Inc. to Kitware for coherence and prevent qtconfig directory name with space/comma.

Re-Hidden Paraview Common Filters following previous update.

Changes in vvMainWindow.cxx are massive, focus on the final form

TODO Updgrade submodules:

  • LVCORE to fix-ruler
  • Common-superbuild to master
  • lidarview-superbuild to master

@nick.laurenson @jerome.dias @melanie.carriere @pierre.dellenbach

Edited by Arnaud Billon

Merge request reports

Loading