Hello Nathan,
you are completely correct. The pl.setCrossMin(crossMin)
is missing (and in order to add it, crossMin
should be a pointer anyway).
This addition would not actually change the behavior of the code since the SubgraphPlanarizer
with the given options is already the default for the PlanarizationLayout
. However, it is definitely confusing.
I will adapt the code for the next release.
Thank you for the report,
Dagobert