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

Combine measures #6

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

Combine measures #6

wants to merge 2 commits into from

Conversation

leehach
Copy link
Contributor

@leehach leehach commented Jun 16, 2018

compactness_measures module was designed with stubs for discrete compactness measures. Discrete measures get_discrete_perim and get_discrete_area were created in geo_discrete_measures module. The body of these functions has been moved into _discrete_perimeter and _discrete_area in compactness_measures. Functions can now be called as:

area(geo) # continuous area
area(geo, geo_cell) # discrete area

perimeter(geo) # continuous area
perimeter(geo, geo_cell) # discrete area

polsby_popper (geo) # continuous Polsby-Popper compactness
polsby_popper (geo, geo_cell) # discrete Polsby-Popper compactness

etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zachlevitt zachlevitt zachlevitt approved these changes

@ykelly ykelly Awaiting requested review from ykelly

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leehach @zachlevitt