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
Patch/cxsmilesgen #221
Patch/cxsmilesgen #221
Conversation
Okay reactions are done, actually turned out existing reactions weren't canonical. Just need to update documentation now which I'll try and get done tonight. |
Looks ok to me, but I did not check all the bit operation stuff, which I am not using daily. John, also, I cannot merge it, because there are conflicts that need manual resolving. Can you plz rebase the branch on the latest master? |
…tset (composed of SmiOpt flags) into the constructor.
…impler and configurable.
…indicate z coordinates.
…s can be canonicalised.
476c90b
to
209208d
Compare
Resolved |
Thanks will get Nina's/Nickolays patch rebased tonight and reviewed properly. |
Work in progress but made SMILES generation options more tunable.