cdk / depict

SMILES Depiction Generator
GNU Lesser General Public License v2.1
54 stars 14 forks source link

The on-line version fails when highlighting is requested on no-kekulisable smiles #9

Closed sauliusg closed 6 years ago

sauliusg commented 6 years ago

The on-line version at http://www.simolecule.com/cdkdepict/depict.html displays nicely the non-kekulisable SMILES like 'n1cccc1', but fails to produce an image when "Highlight" is requested: screenshot from 2018-02-27 17-48-36 Error (exception) message, available at the "broken image" link, was: 'org.springframework.web.util.NestedServletException: Request processing failed; nested exception is java.lang.NullPointerException: Bond order was not set.'. The issue is not reproducible locally, as the downloaded or checked-out and compiled cdkdepict 0.{1,2,3} releases do not parse the 'n1cccc1' on the local machines (see issue #8 ).

johnmay commented 6 years ago

Yes the SMILES is not valid and the structure is in an inconsistent state so things go bananas internally. I've recently relaxed it so you can at least display them but I make no guarentees anything else will work.