ChemistryFeaturization.jl icon indicating copy to clipboard operation
ChemistryFeaturization.jl copied to clipboard

finish updating WeaveModel-related stuff for restructure

Open rkurchin opened this issue 3 years ago • 5 comments

must-haves

  • [ ] get to the point of being able to build/featurize the molecule and convert it to the right format to be fed into WeaveModel.jl
  • [ ] figure out any other naming things, e.g. for BondFeatureDesciptors

nice-to-haves

  • [ ] some kind of visualization, maybe via OpenSMILES or MolecularGraph

Feel free to make a more "granular" version of this checklist, haha

rkurchin avatar Jun 04 '21 18:06 rkurchin

get to the point of being able to build/featurize the molecule and convert it to the right format to be fed into WeaveModel.jl

Does this entail something else in addition to setting up WeaveFeaturization?

some kind of visualization, maybe via OpenSMILES or MolecularGraph

This might be relatively easier to set up (wouldn't a Base.show be enough?)

thazhemadam avatar Jun 04 '21 19:06 thazhemadam

No, I don't think it's really beyond setting that up.

Base.show should have text output in general I think; I was thinking like analogous to the graph visualization we can do for AtomGraph

rkurchin avatar Jun 04 '21 21:06 rkurchin

Well I guess we may also want some convenience stuff for building commonly-desired SpeciesFeatureDescriptors

rkurchin avatar Jun 04 '21 21:06 rkurchin

...which probably involves either bringing back the PyCall to rdkit or maybe playing with that jll

rkurchin avatar Jun 11 '21 18:06 rkurchin

building an AtomGraph from a SMILES string etc. would also be clutch here, and make some parts of the test suite tidier too

rkurchin avatar Jun 23 '21 20:06 rkurchin