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
Feature/advanced sgroup rendering #151
Conversation
…g parts we don't want to show, remapping some atom symbols or adding a label (for zero attach abbreviations).
…d bonds. To hide atoms we take the set complement of the parent atom list in the atom list.
… of each other but actually for Sgroups we want these (carbons) to be hidden.
…ket and they always point the right way.
…as in multiple group Sgroups.
Added some more commits for formatting labels nicely. All done now. |
Okay there was one more. Code to generate them will be in a separate patch but here's a test file with some examples to try,
|
A lot of code, but mostly internal. But looks good. |
Applied and pushed. |
Thanks - yeah realising I needed the label reversing complicate things considerably. |
There seem to be two more regressions: https://jenkins.bigcat.unimaas.nl/job/cdk/lastCompletedBuild/testReport/
|
Darn - does travis not run the tests? Regards, On 13 September 2015 at 07:51, Egon Willighagen notifications@github.com
|
Adds much better rendering - for example >2 brackets and their direction, multiple groups, and mixtures.
Still need to add data display but didn't to figure how that works first.
efficientbits.blogspot.co.uk/2015/09/bringing-molfile-sgroups-to-cdk-demo.html