wouldn't hurt to summarize the logic; otherwise the docstring has little value