Atoms with 8+ neighbors don't draw symmetrically

[xavierholt]

For example, the following SMILES:

N(C)(C)(C)(C)(C)(C)
N(C)(C)(C)(C)(C)(C)(C)
N(C)(C)(C)(C)(C)(C)(C)(C)
N(C)(C)(C)(C)(C)(C)(C)(C)(C)
N(C)(C)(C)(C)(C)(C)(C)(C)(C)(C)

Draw like this:

After a little experimentation, I found that this seems to be an issue with overlap resolution. For example, the molecule with seven carbons (Mol2) draws as expected when overlapSensitivity is 0.27 or higher, but gets ugly at 0.26 and lower. Setting overlapResolutionIterations to zero doesn't change anything...

There might actually be two bugs in here - I'll have to investigate a little more:

• The force-based layout algorithm might be behaving badly?
• There's no way to completely disable overlap resolution?

[xavierholt]

Looks like the resolvePrimaryOverlaps() and resolveSecondaryOverlaps() functions always run once, regardless of the number of overlap resolution steps specified by the user. Only the number of subtree overlap iterations can be controlled by the user at the moment.

• smilesDrawer/src/DrawerBase.js

  Line 617 in 3d5e368

  this.resolvePrimaryOverlaps();

• smilesDrawer/src/DrawerBase.js

  Line 700 in 3d5e368

  this.resolveSecondaryOverlaps(overlapScore.scores);