Skip to content

Project 17 - Urban_Logistic_Demand_Forecasting#1729

Merged
YuvaraniD merged 11 commits intomasterfrom
Yuvarani-playground
Apr 15, 2026
Merged

Project 17 - Urban_Logistic_Demand_Forecasting#1729
YuvaraniD merged 11 commits intomasterfrom
Yuvarani-playground

Conversation

@YuvaraniD
Copy link
Copy Markdown
Collaborator

This project focuses on Urban Logistics Demand Forecasting (Project 17). The work completed so far includes:

  • Performed Exploratory Data Analysis (EDA) to understand data distribution and patterns

  • Conducted data cleaning and preprocessing, including handling missing values and formatting time-based features

  • Performed temporal analysis to study demand trends over time (hourly/daily patterns)

  • Conducted spatial analysis to understand geographic distribution of demand

  • Derived key insights from the dataset to support demand forecasting and decision-making

@YuvaraniD YuvaraniD requested a review from sahithipinnam April 3, 2026 05:54
@YuvaraniD YuvaraniD requested a review from manya0033 April 4, 2026 12:02
Copy link
Copy Markdown
Collaborator

@manya0033 manya0033 left a comment

Choose a reason for hiding this comment

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

Hi @YuvaraniD , great work on this use case! The structure is clear, visualisations are well-labelled with insightful interpretations, and the code is readable. A few things to address before approval:

  1. The data is being downloaded via direct ZIP URLs rather than API v2.1. Could you either confirm with leadership that this is acceptable given the API limitation, or upload CSV versions of the data to the repo as per the checklist requirement for external datasets?
  2. Please resolve the merge conflict in .gitignore so the branch can be merged.

Otherwise, the analysis is well-presented and the insights are clear. Happy to re-review once these are addressed!

@manya0033
Copy link
Copy Markdown
Collaborator

Thanks for uploading the datasets @YuvaraniD ! Just one more thing - I noticed the entire .gitignore file has been deleted in this PR. I think this happened because the old .gitignore had *.zip in it which would have blocked your dataset uploads. Could you restore the .gitignore and instead add a specific exception for the DEPENDENCIES folder.

@YuvaraniD
Copy link
Copy Markdown
Collaborator Author

Hi @manya0033 , I’ve updated the .gitignore file and restored it. Please let me know if anything else needs to be adjusted.

@manya0033
Copy link
Copy Markdown
Collaborator

Hey @YuvaraniD , thanks for updating this! A few small things before I can approve:

There's a small typo on line 8 of the .gitignore - it should just be !datascience/usecases/DEPENDENCIES/**/*.zip (the .DS_Store at the end shouldn't be there). As it is now, it won't actually let your ZIP files through.
Could you also add a few essential rules to the .gitignore? Mainly .ipynb_checkpoints/, pycache/, and .env. These will protect the repo from common junk files, especially Jupyter checkpoint folders which tend to get committed accidentally.
I also noticed there are two notebook files in your folder- one called Project 17_Urban_Logistics_Demand_Forecasting.ipynb (with a space) and one called Project_17_Urban_Logistics_Demand_Forecasting.ipynb (with an underscore). Looks like the old one wasn't removed when you renamed it. Could you delete the one with the space so only the correctly named version remains?

@YuvaraniD
Copy link
Copy Markdown
Collaborator Author

I’ve made the requested changes: Please let me know if anything else is needed @manya0033

@manya0033
Copy link
Copy Markdown
Collaborator

@YuvaraniD Changes addressed, well done. Happy to approve!

@manya0033 manya0033 self-requested a review April 10, 2026 08:08
Copy link
Copy Markdown
Collaborator

@manya0033 manya0033 left a comment

Choose a reason for hiding this comment

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

Changes addressed properly, well done!

@YuvaraniD YuvaraniD merged commit 461d7c5 into master Apr 15, 2026
3 checks passed
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