You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 10, 2019. It is now read-only.
if (children.size() == 1 || children.size() == 2) {
for (S2CellId child : children) {
if (child.isLeaf()) {
cellIds.add(child);
} else {
queue.add(child);
}
}
} else if (children.size() == 3) {
cellIds.addAll(children);
} else if (children.size() == 4) {
cellIds.add(parent);
} else {
assert false; // This should not happen.
}
Image the situation that the rectangle is located near the center of the parent but isn't large enough to cover it, then it can still intersect with all its children.
The right way to do is to first check whether the rectangle contains the parent, and add all its intersected children to the queue.
Example bounds: (46.95913069858797, -123.74038696289062, 48.055558949569566, -121.27670288085938). Queries also returns points in Spokane.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
This snippet of code is problematic:
Image the situation that the rectangle is located near the center of the parent but isn't large enough to cover it, then it can still intersect with all its children.
The right way to do is to first check whether the rectangle contains the parent, and add all its intersected children to the queue.
Example bounds: (46.95913069858797, -123.74038696289062, 48.055558949569566, -121.27670288085938). Queries also returns points in Spokane.
The text was updated successfully, but these errors were encountered: