Skip to content

Assignment 4 #6

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
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

joemCruz
Copy link

Changes to assignment 4

city = None
max_population = 0

for nextFeature in gj_features:
if gj_features[nextFeature]['pop_max'] > max_population:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here you likely do not need gj_features[nextFeature] as the nextFeature in the for loop above should be that value. This is probably why you are getting the errors.

@jlaura
Copy link
Contributor

jlaura commented Feb 27, 2016

@joemCruz Do you want to update and resubmit to fix this line?

@joemCruz
Copy link
Author

Sorry, I was unable to fix the line as I was out for the weekend. If I still can it would be much appreciated.
Thanks,Joseph Cruz

On Saturday, February 27, 2016 1:54 PM, jlaura <[email protected]> wrote:

@joemCruz Do you want to update and resubmit to fix this line?—
Reply to this email directly or view it on GitHub.

@joemCruz
Copy link
Author

Professor Laura,
I am also unfamiliar with the Monte Carlo method that is being used in this section. Any clarification would be much appreciated.
Thanks,Joseph Cruz

On Saturday, February 27, 2016 1:54 PM, jlaura <[email protected]> wrote:

@joemCruz Do you want to update and resubmit to fix this line?—
Reply to this email directly or view it on GitHub.

@jlaura
Copy link
Contributor

jlaura commented Feb 29, 2016

Are you referencing assignment 5?

@jlaura
Copy link
Contributor

jlaura commented Mar 1, 2016

@joemCruz Did you resubmit this to get fixed code upload? If so, respond here and I will update the grade book. Either way is fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants