@qbilius Ok I've done a bunch of what we talked about:
1) integrated with new queue interface
2) largely modularized the logic of the "call" blocks (I've called them "get*") to try to make the logic and default arguments clearer
3) dealt with the issue of the step=0 stuff
4) ... and a variety of other small things.
5) and added a bunch of detail in the tests and debugged the code accordingly.
I have not separated the load and save methods for the DBInterface object into two different subclasses. I'm not yet entirely convinced of the wisdom of doing this. Can we review/merge the existing PR and then possibly deal with that issue later?
@qbilius Ok I've done a bunch of what we talked about:
1) integrated with new queue interface 2) largely modularized the logic of the "call" blocks (I've called them "get*") to try to make the logic and default arguments clearer 3) dealt with the issue of the step=0 stuff 4) ... and a variety of other small things. 5) and added a bunch of detail in the tests and debugged the code accordingly.
I have not separated the load and save methods for the DBInterface object into two different subclasses. I'm not yet entirely convinced of the wisdom of doing this. Can we review/merge the existing PR and then possibly deal with that issue later?