This pull request
- Reorders imports
- Adds Google style docstrings to functions
- Reformats nested whitespace
- Renames global constants to be in UPPER_CASE
Looks good. Thanks for the help! I'm going to try and knock off a few of the bugs that require more in-depth testing soon so I think I'll just merge this in right away if that's ok.
Looks good. Thanks for the help! I'm going to try and knock off a few of the bugs that require more in-depth testing soon so I think I'll just merge this in right away if that's ok.
Also the whole archiving interface is still kind of a proof of concept, and I'm not sure if I'll stick with it or not. The reason I did it that way (enabling line buffering and explicitly closing the log file) is just for flexibility, but there might be a better way of handling it if it gets to be too clumsy.
Also the whole archiving interface is still kind of a proof of concept, and I'm not sure if I'll stick with it or not. The reason I did it that way (enabling line buffering and explicitly closing the log file) is just for flexibility, but there might be a better way of handling it if it gets to be too clumsy.
This pull request
Looks good. Thanks for the help! I'm going to try and knock off a few of the bugs that require more in-depth testing soon so I think I'll just merge this in right away if that's ok.
Also the whole archiving interface is still kind of a proof of concept, and I'm not sure if I'll stick with it or not. The reason I did it that way (enabling line buffering and explicitly closing the log file) is just for flexibility, but there might be a better way of handling it if it gets to be too clumsy.
3e6e90cf87.