search

SpaceGroupUCL / qgisSpaceSyntaxToolkit

Space Syntax Toolkit for QGIS
GNU General Public License v3.0
115 stars 40 forks source link

Values LEN and MD are exchanged in axial analysis [Mac OSX] #143

Closed maurimartins closed 7 years ago

maurimartins commented 7 years ago

Sometimes, the columns LEN and MD are exchanged, affecting results in another measures. This has been observed in axial analysis (SST 0.1.5, QGIS 2.14.3, GDAL/OGR 1.11.4 for Mac OSX 10.11.16 El Captain).

I can not always reproduce this error. All the three maps were verified with SST "Verify" option. Two from three different tested maps presented the error. Would any subtle flaw in linear representation have been misinterpreted by the algorithm, inducing a bug?

I intend to observe more, I'll send you news if I find some error pattern.

Meanwhile, it is recommended that Mac OSX users check the LEN values after run axial analysis.

jorgegil commented 7 years ago

I was able to reproduce by processing an axial map that had already been processed before. In that case the result does not include the "id" column and all columns after that have the wrong label and eventually wrong result.

jorgegil commented 7 years ago

The issue has now been fixed and will be available in the next release. If someone needs a "patch", you can get the DepthmapAnalysis.py file from the latest source and update it in the .qgis2/python/plugins/esstoolkit/analysis folder.