Initiated this issue [change] Change look of traffic charts #366, learned a good amount of stuff about charts and how they work, understood the approach but was stuck, later it was completed by Mentor Gangandeep and I learned most of the things from the approved PR.
Updated the widget, and changed the UI, and added some conditions where users cannot select more than a year or day after today's dates.
Added the functionality where we can select the current 1 day, 3 days, 7 days, 1 week, and 1 year time from the widget itself.
Looked into some functionality for the custom date range and how can we implement that, and added the implementation flow in the flow chart. (attached image below).
Added the functionality for the custom date range on the basis of the span between the start and end date,(need to improve that on the basis of dynamic grouping).
Wrote the logic for dynamic grouping, this would be in models so that the data sent is populated on the basis of grouping.
Added the ajax call in the chart.html on the event apply.daterangepicker, views would have alerts to validate custom range.
Added the functionality for the custom date range on the basis of the span between the start and end date, with getting the data displayed on the x-axis between the start and end date.
Completed the functionality of the dynamic grouping, now the data is grouped for non-defined given custom range and displaced according to every chart.
Improve the PR for some bugs that are there on some requests made to the custom range.
Fix the dashboard, now has some errors with the scripts loading and the widget cannot access the daterangepicker function even though having scripts already on the page.
Would create a lucid flowchart for the Zooming Graphs issue, and would get clear the idea and flow from the mentors.
Would open a PR for Zooming Graphs with the initial implementation of the discussed workflow.
Blockers:
Currently, there are no blockers.
With regards
Yashika
Message has been deleted
yashika jotwani
unread,
Aug 5, 2022, 1:13:31 PM8/5/22
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
Worked on coverage, worked on bugs that came during the review, and fixed the charts during refreshing or loading both on the dashboard and device page.
Would update the zooming PR and look for any sliding event handler on plotly.
Blockers:
Currently, there are no blockers.
With regards
Yashika
yashika jotwani
unread,
Aug 26, 2022, 10:20:37 AM8/26/22
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to OpenWISP Google Summer of Code
[22 Aug – 26 Aug] Project Report
List of tasks I have worked on:
Worked on the failing test for the PR(date range). I was blocked, I updated the tests and they were working fine individually but failing in running during all the tests.
Started working on API PR until I was blocked. I have made some research and started debugging the implementation with reference of my proposed solution.
Worked on the requested changes for the PR(date range): Progress: -
Would update the zooming PR and look for any sliding event handler on plotly.
Blockers:
I am working on the requested changes, and it might have some issues but I will work on it.
I am not able to figure out how we can remove the CDN link from `chart.html` and successfully load the date range picker, I tried it multiple times but am stuck on that. As was suggested by Gagandeep to add all the `min.js` files locally and add them. It works on the `home` page but not loading on the `device` page.
[P.S.] : I was not able to attend the recent demo meet because I had urgent travelling on Wednesday, I tried to work but there was no stable internet connection.