-
Notifications
You must be signed in to change notification settings - Fork 18
assignment_05 #5
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
Open
thervig
wants to merge
37
commits into
Geospatial-Python:master
Choose a base branch
from
thervig:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
d33fe1e
Update utils.py
thervig 81ba330
Update io_geojson.py
thervig f829fc8
Update test_utils.py
thervig ea3dc58
Update analytics.py
thervig dd7c86b
Update test_analytics.py
thervig 15e2a1b
Update utils.py
thervig 7a9dc1e
Update analytics.py
thervig 9853c1c
Update analytics.py
thervig 730de3e
Update functional_test.py
thervig 1551581
Update analytics.py
thervig 5ce2047
Update utils.py
thervig bb6cede
Update analytics.py
thervig b4d1048
Update functional_test.py
thervig 0da10c4
Update test_analytics.py
thervig b9a2f7e
Update test_analytics.py
thervig bd6eb95
Update test_analytics.py
thervig 97703a4
Update utils.py
thervig 6b5e221
Update test_utils.py
thervig c2c921b
Update test_utils.py
thervig 6c54633
Update analytics.py
thervig c418d05
Update test_analytics.py
thervig 1d22835
Update test_analytics.py
thervig 6bf407c
Update test_analytics.py
thervig cddc291
Update utils.py
thervig e073788
Update functional_test.py
thervig e9b167f
Update analytics.py
thervig 8c8869a
Update test_analytics.py
thervig 480688f
Update functional_test.py
thervig 5d134f6
Update functional_test.py
thervig 47ea217
Update test_utils.py
thervig 57f7b1d
Update functional_test.py
thervig af8c987
Update test_analytics.py
thervig 49fd9cb
Update test_analytics.py
thervig 8a45d70
Update functional_test.py
thervig 516475d
Update analytics.py
thervig 989e957
Update functional_test.py
thervig b5d8bea
Update test_analytics.py
thervig File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,97 @@ | ||
import math | ||
import random | ||
from .utils import euclidean_distance | ||
from .utils import generate_random | ||
|
||
|
||
def find_largest_city(gj): | ||
""" | ||
Iterate through a geojson feature collection and | ||
find the largest city. Assume that the key | ||
to access the maximum population is 'pop_max'. | ||
Parameters | ||
---------- | ||
gj : dict | ||
A GeoJSON file read in as a Python dictionary | ||
Returns | ||
------- | ||
city : str | ||
The largest city | ||
population : int | ||
The population of the largest city | ||
""" | ||
|
||
temp = 0 | ||
city = "" | ||
max_population = 0 | ||
for i in gj['features']: | ||
if i['properties']['pop_max'] > temp: | ||
temp = i['properties']['pop_max'] | ||
city = i['properties']['name'] | ||
max_population = temp | ||
|
||
return city, max_population | ||
|
||
|
||
def average_nearest_neighbor_distance(points): | ||
""" | ||
Given a set of points, compute the average nearest neighbor. | ||
Parameters | ||
---------- | ||
points : list | ||
A list of points in the form (x,y) | ||
Returns | ||
------- | ||
mean_d : float | ||
Average nearest neighbor distance | ||
References | ||
---------- | ||
Clark and Evan (1954 Distance to Nearest Neighbor as a | ||
Measure of Spatial Relationships in Populations. Ecology. 35(4) | ||
p. 445-453. | ||
""" | ||
sum_nn_dis = 0 | ||
|
||
for point_1 in points: | ||
first = True | ||
for point_2 in points: | ||
if point_1 == point_2: | ||
continue | ||
else: | ||
distance = euclidean_distance(point_1, point_2) | ||
if first: | ||
nn_dis = distance | ||
first = False | ||
elif distance < nn_dis: | ||
nn_dis = distance | ||
|
||
sum_nn_dis += nn_dis | ||
mean_distance = sum_nn_dis / len(points) | ||
return mean_distance | ||
|
||
def permutations(p = 99, n = 100): | ||
permutations = [] | ||
for i in range(p): | ||
points_list = generate_random(n) | ||
permutations.append(average_nearest_neighbor_distance(points_list)) | ||
|
||
return permutations | ||
|
||
|
||
def critical_points(permutations): | ||
""" | ||
Lowest distance and greatest distance of points. | ||
""" | ||
|
||
lower = min(permutations) | ||
upper = max(permutations) | ||
return lower, upper | ||
|
||
|
||
def significant_distance(lower, upper, observed): | ||
if (lower > observed) or (upper < observed): | ||
result = True | ||
else: | ||
result = False | ||
|
||
return result |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
import json | ||
|
||
def read_geojson(input_file): | ||
""" | ||
Read a geojson file | ||
Parameters | ||
---------- | ||
input_file : str | ||
The PATH to the data to be read | ||
Returns | ||
------- | ||
gj : dict | ||
An in memory version of the geojson | ||
""" | ||
# Please use the python json module (imported above) | ||
# to solve this one. | ||
with open ('data/us_cities.geojson', 'r') as f: | ||
gj = json.load(f) | ||
|
||
return gj |
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
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
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
Oops, something went wrong.
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.
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 looks pretty good - see my comment last week for comparing
points1 == points2
. Make sure that this function returns something. Right now it does not - so the test fails.