Another round of change detection v1 notebook updates. #152
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Feedback from Jonathan Evens originally for the changes.
| "In the building construction feed, a change result contains a geographic polygon or grid cell that represents the footprint of a contiguous area where we have detected building construction. Thus, these polygons represent general built-up area, and may not identify individual buildings. They may be a group of small buildings clumped together, or a new adjacent wing of an existing building.\n", | ||
| "\n", | ||
| "In the case of the road construction feed, the change results contain a linestring, instead of a polygon. This linestring represents the skeleton of a road segment that has been constructed. \n", | ||
| "In the case of the road construction feed, the change results also contain a polygon or grid cell. It represents the footprint of a contiguous area where a road segment has been constructed.\n", |
Contributor
There was a problem hiding this comment.
Can we share any more detail about what constitutes a "road segment" here?
| "#subscription_id = \"dd2878b3-eaf0-4cff-b7f9-7a39d6f460a5\" # Monthly Building Construction\n", | ||
| "subscription_id = \"dbfcd806-b44d-45d5-b84d-3566a80eca27\" # Monthly Road Construction\n", | ||
| "#subscription_id = \"635c57d3-b921-453f-8e79-70929e50a470\" # Monthly Building Construction\n", | ||
| "subscription_id = \"8869cf69-9596-47ab-8d94-52b94fcf528e\" # Monthly Road Construction\n", |
Contributor
|
LGTM! Nice digram explaining cadence and lag. Adding an animation might be useful, too. |
strixcuriosus
approved these changes
Sep 18, 2020
Contributor
Author
|
Thanks @strixcuriosus for the feedback! We are trying to keep this PR really focused and bring all this Jupyter notebook work to a close, so holding off on any extensive changes for now like a new animation so that we can get this out the door. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Feedback from Jonathan Evens originally for the changes.