Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Shared library #21

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Shared library #21

wants to merge 7 commits into from

Conversation

jamesshannon
Copy link

@jamesshannon jamesshannon commented Nov 4, 2020 •
edited
Loading

Created some shared library classes. This should greatly simplify adding new counties.

I want to propose this PR so that it can be reviewed. Don't merge quite yet -- I'm still testing it against Placer and might make some tweaks to cptlib.

Closes #18

@jamesshannon jamesshannon marked this pull request as ready for review November 4, 2020 20:26
@jamesshannon
FIxex
f520ee5 
* Mixed up lat/long based on x/y from centroid
* Write CSV header
@jamesshannon
Stuff
decab81 
* Added length property to each Parcels generator
* Added % status to parser method
* Added test_limit to parser method for testing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create reusable Scraping and Parsing classes
1 participant
@jamesshannon