Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

In memory identity template library. #41

Closed
wants to merge 8 commits into from

Conversation

johnmay
Copy link
Member

@johnmay johnmay commented Aug 16, 2014

One piece towards de facto 2D layout of rings with the structure diagram generator.

…representations when every except the element type is removed.
…-util' but not committed yet (depend on this patch).
…ng placed horizontal. By setting the default bond vector as static we can check if the caller has really provided a vector to use. If the default was provided we can now still place the chain horizontal.
…ylic groups are always placed with the carbonyl pointing up or down when at the end of a chain.
@johnmay
Copy link
Member Author

johnmay commented Aug 18, 2014

I've added the actual usage now. This means rings such as caffeine, steroids, etc, etc, are laid out nicely.

I used ChEBI to create the template library and will add the code to 'cdk-build-util' once this is added (need the 'skeleton' method). Some manual tweaks were needed so an improved library that more people have looked at would be better.

I've also fixed some other issues with the SDG, the comments indicated chains should be placed horizontally but this was not happening (example). Was quite a simple fix.

@johnmay johnmay closed this Sep 9, 2014
@johnmay johnmay deleted the feature/template-ident-library branch September 9, 2014 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant