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

NEW FEATURE PR- Inclusion of data extraction for both festival dataset 2014-2016 and 2017 - 2019 September #9

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

lazy-whatever
Copy link

Description

  • Included the README for two main datasets used for data correlation analysis.
  • Included several extraction and data cleaning scripts to obtain both festival dataset - 2014-2016 and 2017 - 2019 September.

Related Issue

Closes Festival Section of Issue #2

Summary of Changes

  • New Content Added:
    • Created "festival" folder (and all new scripts and README.md inside it)

Testing

  • [ ✓] Previewed documentation
  • [✓ ] Verified content accuracy

Checklist

  • [✓ ] Code comments/documentation updated
  • [ ✓] Relevant files updated
  • [ ✓] External links verified (if any)

Copy link
Contributor

@RattataKing RattataKing left a comment

Choose a reason for hiding this comment

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

Thanks! File structure looks good to me, need @LindyZh to check if the final cleaned csv format matches with her expectation. NIT: maybe add some words to explain why festival data is grouped into 2014-2016 & 2017-future.

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