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

improve/expand tests for FeaturizedAtoms, encoding

Open rkurchin opened this issue 3 years ago • 2 comments

They also probably need to be reorganized a bit in light of the addition of the FeaturizedAtoms type...

rkurchin avatar Jun 30 '21 17:06 rkurchin

On a slightly related note, the documentation may also need some reorganization.

thazhemadam avatar Jun 30 '21 19:06 thazhemadam

renamed issue because realized we also need tests in the AtomGraph tests that the keyword arguments work. Currently those are only tested in the utils tests for graph-building, but then when a semicolon got lost in the AtomGraph constructor such that all keyword arguments accidentally became positional, it didn't break anything...

rkurchin avatar Jul 01 '21 19:07 rkurchin