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

Bus cost #1007

Merged
merged 21 commits into from
Jan 27, 2024
Merged

Bus cost #1007

merged 21 commits into from
Jan 27, 2024

Conversation

csuyat-dot
Copy link
Contributor

@csuyat-dot csuyat-dot commented Jan 27, 2024

2nd PR to #897

On FTA_bus_grant_analyis and tircp_bus_analysis notebooks, was able to clean and add new columns for prop_type and bus_size_type. Also deprecated all the analysis cells as the analysis will be moved to the cost_per_bus_analysis notebook.

On cost_per_bus_analysis notebook, was able to read in both dataset. Calculated a new cost_per_bus column. Updated the summary cells with more descriptive variable names. Consolidated the chart cells by making a function that takes in chart params. Also reworked the statistical analysis (std_dev and z-score) for cost per bus.

However, noticed there were none values in prop_type. Dug into the FTA notebook again and see there are 50+ rows with none prop types that need adjusting (the cleaning step did not account for rows that did not have "( )".)

So far, created a sub-df of only the none value rows, ran it against a list of keywords to replace the none values. Have a couple of rows left to check.

next iteration will remove the none value rows from the initial df, then append the new rows.

@csuyat-dot csuyat-dot merged commit 88ef085 into main Jan 27, 2024
3 checks passed
@csuyat-dot csuyat-dot deleted the bus_cost branch January 27, 2024 00:13
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.

1 participant