Untangle VGA from sala/depthmapX/cli and move functionality to a module. More specifically the move is of the "Run Visibility Graph Analysis..." menu item and related dialog (renamed from OptionsDlg to VGAOptionsDlg). This dialog contains the all-cell analyses Isovist (Calculates isovist properties from each cell), Local and Global Visibility, Metric and Angular mean depths, and Through-vision. The move also includes the three "Depth" menu items (Visual, Angular Metric) which calculate the respective depth from a cell, but also cli and regression tests (there are no VGA core tests).
If there is agreement with the spirit of this change I will also convert the other analyses (axial, segment, agents) as modules.
For the most straightforward comparison only refer to changes of the first commit, as the second commit auto-formats code.
Untangle VGA from sala/depthmapX/cli and move functionality to a module. More specifically the move is of the "Run Visibility Graph Analysis..." menu item and related dialog (renamed from OptionsDlg to VGAOptionsDlg). This dialog contains the all-cell analyses Isovist (Calculates isovist properties from each cell), Local and Global Visibility, Metric and Angular mean depths, and Through-vision. The move also includes the three "Depth" menu items (Visual, Angular Metric) which calculate the respective depth from a cell, but also cli and regression tests (there are no VGA core tests).
If there is agreement with the spirit of this change I will also convert the other analyses (axial, segment, agents) as modules.
For the most straightforward comparison only refer to changes of the first commit, as the second commit auto-formats code.