Open danielhollas opened 2 years ago
This turned out to be rather annoying, looks like the methods in rdkit are rather unstable, or I am doing something wrong.
To reduce the scope of this, we will aim for stable results for a minimum test set of molecules defined in #4, in conjunction with #29 and #28 which will allow users to upload / remove individual conformers themselves in case automatic generation fails.
I will also try to be a good citizen and upstream general improvements to aiidalab-widgeta-base.
An approach to look into used in ternviz
https://github.com/whitead/ternviz/blob/2a34ba5be73d8aac2e192d00a4af4e5e365cbd36/ternviz/lib.py#L127
Also SMILES validation and canonicalization: https://github.com/whitead/ternviz/blob/2a34ba5be73d8aac2e192d00a4af4e5e365cbd36/ternviz/lib.py#L23
Code is under MIT license.
Take a look at example 7 here: https://www.programcreek.com/python/example/110781/rdkit.Chem.AllChem.EmbedMolecule
Possibly an improvement to the SmilesWidget in aiidalab-widget-base. But need a good test set for this.