search

Electrostatics / APBS_Sphinx

APBS Rewrite
http://www.poissonboltzmann.org
1 stars 3 forks source link

Sphinx: Unit testing implementation #3

Open lizutah opened 9 years ago

lizutah commented 9 years ago

add unit tests to multiprocess branch and merge with master

keith923 commented 9 years ago

There is a lot of technical debt to be payed here. It's all my fault, and it's mostly from the high-speed code-a-thon that happened around April or May to get a working prototype. This included reading the pipeline file, and running geoflow.

We are going to put this off some more, and maybe have Juan work on it. Another reason for putting it off is that I'm not entirely convinced that we want to link the plugins the way that we are using __getattr__, and using Python as the language for our pipeline files.

We should have a discussion with @kmonson, @erikjensen, myself, and anyone else interested, to discuss how it's done now, and see if there are merits to creating a DSL for the pipeline file. Likely as not, once we get more plugins working, we'll have a better idea of our needs.

Until then, this issue is on hold.

keith923 commented 9 years ago

I'd like to see us have Travis CI setup for Sphinx as part of this issue.