Skip to content

Project27: smart parking demand prediction#1922

Open
shreerai001 wants to merge 43 commits into
masterfrom
project27-smart-parking-demand-prediction
Open

Project27: smart parking demand prediction#1922
shreerai001 wants to merge 43 commits into
masterfrom
project27-smart-parking-demand-prediction

Conversation

@shreerai001
Copy link
Copy Markdown
Collaborator

Summary

This PR merges the Smart Parking Demand Prediction project into master.

Included

  • Notebook updates for smart parking demand analysis
  • Data cleaning and feature engineering
  • Model training and evaluation
  • Updated conclusions and project documentation

Validation

  • Notebook runs end to end
  • Model evaluation cells execute successfully
  • Final report text was updated to reflect the validated workflow

Copy link
Copy Markdown
Contributor

@molliefernandez-mentor molliefernandez-mentor left a comment

Choose a reason for hiding this comment

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

Hi Shree, as discussed:

  • Please rename your use case to comply with naming conventions using the Usecase Naming tool
  • Please make sure you rebase and repush your pull request as there are some deletions of newly added use cases in your PR.
  • There is one error and one visualisation which is not displaying

@shreerai001 shreerai001 force-pushed the project27-smart-parking-demand-prediction branch from 5857dca to 31320d1 Compare May 19, 2026 18:13
@shreerai001
Copy link
Copy Markdown
Collaborator Author

Hi Mollie, thanks for the feedback. I have now rebased and updated the PR, restored the unintended deletions, updated the use case naming, and fixed the remaining visualization/error issue. Please let me know if any further changes are required.

Copy link
Copy Markdown
Contributor

@molliefernandez-mentor molliefernandez-mentor left a comment

Choose a reason for hiding this comment

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

Hi Shree, just a few changes from me:

  • Please make sure you are using Australian English
  • The is an error under the data ingestion section
  • I would love to see more explanation under the numbered sections in the exploratory data analysis and the machine learning development model section, about what you are doing and why and what it shows.
  • Looks like the high demand zone map isn't working

@shreerai001
Copy link
Copy Markdown
Collaborator Author

Hi Mollie, thanks for the feedback. I have now updated the PR according to your comments, Please let me know if any further changes are required. Thank you!

Copy link
Copy Markdown
Contributor

@molliefernandez-mentor molliefernandez-mentor left a comment

Choose a reason for hiding this comment

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

Hey Shree, remember that your playground folder needs to be deleted at the end of the trimester. So don't leave any files in there that are needed for your use case.

Just one thing on the use case, can you please remove "executive summary" and "use case" from the title parts of the notebook.

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.

7 participants