Patch/rxnlayout #357

Merged
merged 3 commits into from Aug 23, 2017

Conversation

Projects
None yet
2 participants
@johnmay
Member

johnmay commented Aug 22, 2017

Some improvements to rxn layout #270, still need to do more and fix the Markush connection issues but some progress here

Alignment

Before:
image

After:
image

Less aggressive mapping

I've left in the code for the more aggressive alignment as may still be useful

Before:
image
After:
image

johnmay added some commits Aug 22, 2017

Fix problem with placing positional variation, rather than skipping t…
…o the next one we initially pre sort the bonds in the order we prefer.
@johnmay

This comment has been minimized.

Show comment Hide comment
@johnmay

johnmay Aug 22, 2017

Member

Also fixed this now,

Before:
image

After:
image

Member

johnmay commented Aug 22, 2017

Also fixed this now,

Before:
image

After:
image

@egonw egonw merged commit ac6a353 into master Aug 23, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@johnmay johnmay deleted the patch/rxnlayout branch Mar 18, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment