Stem devs,<br><br>This is a review of Stem commits from 2012-12-11 through 2012-12-21.<br><br>Damian, you are very good at documentation, concise and informative.  I tend to hit one or the other, but not both.  I recognize when someone gets the right mix.  Good work.<br>
<br>I, too, have found logging.basicConfig[0] hides too much about how it works and, now, I roll my own logging handler early.<br><br>Would GETINFO/GETCONF cache hit logging[1] benefit from log_once?  Or even something between log and log_once (e.g. rate_limited_log)?<br>
<br>Is there any particular reason for moving away from <a href="http://readthedocs.org">readthedocs.org</a>?  I'm curious if there was more than a desire to self-host as much as possible.<br><br>Good work on the Controller.get_conf() clean-up[2].  This is easier to code against with the predictable return types.  I have already written a new method with this and always getting a list (event empty) reduces the amount of error checking I had to do.<br>
<br>The heartbeat time tracking is a good idea[3].  But, I wonder, should this bother with an accessor method?  Could this work as just well as an (non-callable) attribute?<br><br clear="all">[0]: <a href="https://gitweb.torproject.org/stem.git/commit/a7b275a3d64301da4d23137dbadd4842cc7a041f">https://gitweb.torproject.org/stem.git/commit/a7b275a3d64301da4d23137dbadd4842cc7a041f</a><br>
[1]: <a href="https://gitweb.torproject.org/stem.git/commit/9b5ff19be35ebbf8c75772a63406ba23762b37de">https://gitweb.torproject.org/stem.git/commit/9b5ff19be35ebbf8c75772a63406ba23762b37de</a><br>[2]: <a href="https://gitweb.torproject.org/stem.git/commit/5b45d3125c811067d7c3ba0a11e324423c88d925">https://gitweb.torproject.org/stem.git/commit/5b45d3125c811067d7c3ba0a11e324423c88d925</a><br>
[3]: <a href="https://gitweb.torproject.org/stem.git/commit/6bc92816dc4356a204fdde0f160ee344875143a3">https://gitweb.torproject.org/stem.git/commit/6bc92816dc4356a204fdde0f160ee344875143a3</a><br><br>-- <br>Sean Robinson<br>
<br>