-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
processing chapter #90
Conversation
It would be good to have another author to pass over this and probably do some integration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've done some local review, and have some of these changes myself on my spatial data processing branch. I've run into issues using the new data to complete this chapter, and I've outlined where I hit these issues in the text.
|
||
```python | ||
|
||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be removed
sheds.plot(ax=ax, edgecolor='grey', column='lintensity', legend=True, | ||
scheme='fisher_jenks', | ||
classification_kwds={'k':10}) | ||
#cafes.plot(ax=ax, c='purple', alpha=0.5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of these repeated comments need to be removed.
sheds.plot(ax=ax, edgecolor='grey', column='area', legend=True, | ||
scheme='fisher_jenks', | ||
classification_kwds={'k':10}) | ||
#cafes.plot(ax=ax, c='purple', alpha=0.5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comments need to be removed.
# Display airports on top | ||
#listings.plot(ax=ax, c='green') | ||
sheds.plot(ax=ax, alpha=.7, edgecolor='white', column='listings', legend=True, scheme='quantiles') | ||
#cafes.plot(ax=ax, c='purple', alpha=0.5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove comments.
# Display airports on top | ||
#listings.plot(ax=ax, c='green') | ||
sheds.plot(ax=ax, alpha=.7, edgecolor='white') | ||
cafes.plot(ax=ax, c='purple', alpha=0.5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove comments
ax.imshow(img, extent=ext) | ||
# Display airports on top | ||
sd_tracts.plot(ax=ax, alpha=0.6, edgecolor='lightgrey', | ||
column='price', legend=True, scheme='quantiles') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove comments.
ax.imshow(img, extent=ext) | ||
# Display airports on top | ||
sd_tracts.plot(ax=ax, alpha=0.6, color='green',edgecolor='lightgrey') | ||
#listings.plot(ax=ax, c='green') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove comments.
state of California: | ||
|
||
```python | ||
tracts = '../data/sandiego/tl_2019_06_tract.shp' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file is no longer provided. Further, using the acs_tracts_clean.shp
data results in some screw-ups.
- The tracts don't need to be filtered anymore
- The CRS seems to be incorrectly set? I can't seem to get the two to agree on position correctly with a cursory investigation.
This now has all the references converted to the new jupyterbook format. |
No description provided.