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.

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

exercise01_Alex #23

Closed
wants to merge 0 commits into from
Closed

exercise01_Alex #23

wants to merge 0 commits into from

Conversation

asmcroberts
Copy link
Contributor

No description provided.

@ncsg ncsg deleted a comment from review-notebook-app bot Feb 10, 2025
@ncsg ncsg deleted a comment from gitnotebooks bot Feb 10, 2025
Copy link
Contributor

@chesterharvey chesterharvey left a comment

Choose a reason for hiding this comment

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

Great

Copy link
Contributor

Choose a reason for hiding this comment

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

# Exercise 01
# How many times did each type of request get made?
    # Need: Type of request [Description]

I like how you structured everything, its easy to understand

Copy link
Contributor

Choose a reason for hiding this comment

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

    #3. Print results

request_type_counts = {}
city_counts = {}

#Requests by Type
for request in service_requests:

Here's where it gets a little confusing for me not gonna lie

Copy link
Contributor

Choose a reason for hiding this comment

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

        request_type_counts[request_type] = 1

#Requests by City
for request in service_requests:
    city = request['City']
    if city in city_counts:
        city_counts[city] += 1
    else:

ok good

Copy link
Contributor

Choose a reason for hiding this comment

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

        city_counts[city] = 1

print("Requests by Type:")
print(request_type_counts)

print("Requests by City:")
print(city_counts)

straight forward and got the job done

Copy link
Contributor

@Huckleberrypablo Huckleberrypablo left a comment

Choose a reason for hiding this comment

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

pretty straight forward, did what was asked. Good job

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.

4 participants