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

Task 4 - INCOMPLETE #45

Closed
wants to merge 7 commits into from
Closed

Task 4 - INCOMPLETE #45

wants to merge 7 commits into from

Conversation

ryantsui1
Copy link

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Adding the histogram for # of incidents over years within the TPS dataset.

What did you learn from the changes you have made?

There is a slow upward trend for auto theft before a spike in 2022,2023 and a high mid-year number for 2024.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

DO NOT MERGE AS IS. I've tried to make a vis to show that there was no premises type that was majorly contributing to the auto thefts. However, got stuck as the Auto Theft Type is built on T/F on their own separate columns.

Kept some work in there to share the idea but opted out of completing it because T/F alternates depending on whether the first occurence in the list is a True or False + I have no idea how to fix it.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

Checklist

  • I can confirm that my changes are working as intended

Copy link

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

@ryantsui1 ryantsui1 closed this Aug 10, 2024
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.

3 participants