perses
perses copied to clipboard
set aromatic draw style to `OEAromaticStyle_Circle` in atom mapper
Description
Motivation and context
Resolves #1082
How has this been tested?
Change log
Set aromatic draw style to `OEAromaticStyle_Circle` in atom mapper rendering
Codecov Report
Merging #1103 (ed5bb3a) into main (2ff4a7b) will increase coverage by
0.00%
. The diff coverage is100.00%
.
Thanks! I was trying to see how the visualization changes using the code in https://gist.github.com/IAlibay/089f614665ab3a10d076990ba8cd8024 from issue #1078 , but I don't really see a difference. What do we expect to be different here? As far as I remember that notebook/issue was the cause of this feature request.
I think our answer is somewhere here: https://github.com/choderalab/perses/blob/main/perses/rjmc/atom_mapping.py#L305-L310
Is calling oedepict.OERenderMolecule(oeimage, display)
enough to get all the opts
we configured, or do we need to pass a return value to another function?
This failure looks unrelated, so merging.