Skip to content
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

improve/expand tests for FeaturizedAtoms, encoding #94

Open
rkurchin opened this issue Jun 30, 2021 · 2 comments
Open

improve/expand tests for FeaturizedAtoms, encoding #94

rkurchin opened this issue Jun 30, 2021 · 2 comments
Labels
priority feature to add or bug to fix soon!

Comments

@rkurchin
Copy link
Member

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

@rkurchin rkurchin added the priority feature to add or bug to fix soon! label Jun 30, 2021
@rkurchin rkurchin added this to To do, fairly soon in ChemistryFeaturization Jun 30, 2021
@thazhemadam
Copy link
Member

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

@rkurchin rkurchin changed the title better tests for FeaturizedAtoms, encoding, etc. improve/expand tests for FeaturizedAtoms, encoding, AtomGraph Jul 1, 2021
@rkurchin
Copy link
Member Author

rkurchin commented Jul 1, 2021

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...

@thazhemadam thazhemadam changed the title improve/expand tests for FeaturizedAtoms, encoding, AtomGraph improve/expand tests for FeaturizedAtoms, encoding Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority feature to add or bug to fix soon!
Projects
ChemistryFeaturization
To do, fairly soon
Development

No branches or pull requests

2 participants