Include JSON dataset in the demo-project

SajidAlamQB commented 1 month ago

Description

Development notes

Replaced companies.csv with companies_json.json and added a new node that converts the json into a csv. This should make the json pre-viewable in the demo.

This is what it looks like with the new companies_json.json in the view

Edit:

After reviewing with Rashida we now have a new reporting node get_top_shuttles_data that takes the head of the input table and converts it into JSON. See below for results:

Checklist

[x] Read the contributing guidelines
[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[x] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

ravi-kumar-pilla commented 1 month ago

Hi @SajidAlamQB ,

I see that #1929 suggests to replace companies to json. I think it would be nice to introduce a new dataset rather than modifying existing datasets in the demo_project.

Thank you

SajidAlamQB commented 1 month ago

I see that #1929 suggests to replace companies to json. I think it would be nice to introduce a new dataset rather than modifying existing datasets in the demo_project.

I'm not opposed but I feel like adding another dataset would be a bit random. If we wanted to include it for this demo we would have to rework it to make it feel natural.

SajidAlamQB commented 1 month ago

After reviewing with Rashida we now have a new reporting node get_top_shuttles_data that takes the head of the input table and converts it into JSON. See below for results:

kedro-org / kedro-viz