Added Tutorial section to explain DynamicMap groupby methods #667
Conversation
|
Great! Thanks for updating the docs! Ready to merge? |
|
Ready. |
| { | ||
| "cell_type": "markdown", | ||
| "metadata": {}, | ||
| "source": [ |
There was a problem hiding this comment.
Line 714 below says "need to check" -- what's the answer to that question?
There was a problem hiding this comment.
Good question, looks like @jlstevens forgot to address his own question 😄 I think the statement is correct as is, as long as the dimension is unbounded on either end it will default to open mode.
There was a problem hiding this comment.
Ok, I'll remove the qualifier while I'm doing an editing pass on this tutorial to clean up the new changes.
There was a problem hiding this comment.
In that case we can remove that stray comment...oops!
Edit: Thanks Jim for cleaning it up and giving it another pass!
There was a problem hiding this comment.
Would "At least one of the declared key dimensions must have an unbounded range (i.e., with no upper bound specified)." be correct?
There was a problem hiding this comment.
I think so...though Philipp suggests it works if either bounds is None, including the lower bound? I just wonder how the scrubber handles that...does it decrease starting from the upper bound?
There was a problem hiding this comment.
It just ignores all the bounds, at present; all that matters is that one bound is not specified. I've clarified it to say that, but we're still deciding on the larger issue of how to describe these various modes.
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
No description provided.