theochem/Selector

[Utils] Utils.py has unfinished to-do and lacks testing

Closed this issue · 4 comments

The utils.py has a large to-do block here Is implementation of this is still needed?

Also, mol_loader() needs tests written for it in test_util.py

I think that @FanwangM may already have code to do this, probably in a different repository for a different project.

I can also imagine outsourcing this to an external package, and just require the user to handle the input. I think it's a good question for @FanwangM

This add-on should be demonstrated with a Jupyter Notebook, and not part of the main package I think.

I think it's safe to move it to Jupyter Notebook, which will be converted into static website pages later. @AWBroscius

Sorry for the late reply.