[Weekly Progress Report GSoC 2025] Improve OpenWISP General Map

[Deepanshu Sahu]

Hello everyone,

This thread will be used to track the progress of the GSoC 2025 project, Improve OpenWISP General Map.

[May 19 - May 23]

List of Tasks I have worked on :

• Rest Api Revision PR is ready for review.
  
• Work on Switch to ChannelsLiverServerTestCase in openwisp-utils PR.
• Created a sample app, need to set up selenium tests for the Django channels PR.
  

Tasks I will be working on next week:

• Complete Django channels PR
  
• Complete Rest Api Revision PR
• Complete switch to ChannelsLiverServerTestCase in openwisp-utils PR.
  
• Work on the floorplan endpoint PR
• Raise PR for #561

Blockers:

• Currently, there are no blockers

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[May 26 - May 30]

List of Tasks I have worked on :

• Django channels PR is ready for review.
  

• Work on Switch to ChannelsLiverServerTestCase in openwisp-utils PR.

• Made points to discuss ui adjustments.

Tasks I will be working on next week:

• Address comments in any open PR

• Complete switch to ChannelsLiverServerTestCase in openwisp-utils PR.
  
• Work on the floorplan endpoint PR
• Raise PR for #561

Blockers:

• No blockers

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[June 2 - June 6]

List of Tasks I have worked on :

• Reviewed the Figma design and made points for improvements.
• Test the django channels fix in the openwisp-utils PR
  
• Addressed comments on reversion PR
• Created a draft PR for adding a dedicated map page

Tasks I will be working on next week:

• Address comments in any open PR

• Complete Add dedicate map page PR
• Complete Foorplan endpoint PR

Blockers:

• No blockers

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[June 9 - June 13]

List of Tasks I have worked on :

• New Map page PR is ready for review (needs discussion with the UI team)
  
• Created a PR to add a new icon for the Map page (needs discussion with the UI team)
• Created a draft PR to add the floorplan map above the dashboard geo map.
• Almost completed with the floorplan endpoint PR, only minor code restructuring remains.
• Addressed comments in the channels PR

Tasks I will be working on next week:

• Address comments in any open PR

• Test the floorplan map PR with data coming from the floorplan endpoint PR and prepare both for review.
  
• Make these 2 PRs deploy in staging.

Blockers:

• No blockers

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[June 16 - June 20]

List of Tasks I have worked on :

• [feature] Add API endpoint for indoor map coordinates #976 is ready for review.
• [feature] Support custom CRS in default config #381 is ready for review.
  
• [feature] Support custom CRS in default config #381. This PR also addresses an issue with the zoom buttons not disabling when there are float-type zoom values and added units, and browser tests for all cases.
• [feature] Add floorplan map above dashboard map #680 is almost done, need to improve ui a little and add tests

Tasks I will be working on next week:

• Address comments in any open PR

• Complete [feature] Add API endpoint for indoor map coordinates #976 and [feature] Add floorplan map above dashboard map #680

Blockers:

• No blockers

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[June 23 - June 27]

List of Tasks I have worked on :

• [feature] Add API endpoint for indoor map coordinates #976 is ready for review.
  
• [feature] Support custom CRS in default config #381 is ready for review.

• feature] Add floorplan map above dashboard map #680, addressed the comments, and opened for review
  Completed the UI and covered the edge cases.Need to address some CSS issues after integrating netjsongraph.js build files. and add tests

Tasks I will be working on next week:

• Address comments in any open PR

• Implement ui from Figma design

Blockers:

• No blockers

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[June 30 - July 04]

List of Tasks I have worked on :

• [feature] Support custom CRS in default config #381 is merged.

• [feature] Add API endpoint for indoor map coordinates #976 is ready for review.

• [feature] Add floorplan map above dashboard map #680, implemented all the things discussed in a one-on-one session with Gagan.
  
  •  Moved the data restructuring required to render the netjsongraph map to the frontend.
  • Make the endpoint show the least positive floor, or if there isn't any, then show the max negative floor
  • Floors are rendering correctly for all negative, all positive, and mixed floors.
  • Create a new div element for rendering the floor for the first time, and hide it when switching

Tasks I will be working on next week:

• Address comments in any open PR

• There is a bug introduced when adding 2 netjsongraph overlapping instances. The zoom button of both the geo map and the indoor map started pointing to the indoor map, even after removing the indoor map from the DOM, the zoom functionality remains from the indoor map.
  Need to find a fix for this.

• Implement ui from Figma design

Blockers:

• No blockers

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[July 07 - July 11]

List of Tasks I have worked on :

• [feature] Add API endpoint for indoor map coordinates #976 is ready for review.

• • Recreated the filtering logic as multi-tenant support was missing.
  • Added all test scenarios mentioned in the PR.
  • Added docs.

• [feature] Add floorplan map above dashboard map #680, 

• • Make navigation of floors to show 5 at a time with positve ones at top and negative ones at bottom.
  • Tried to fix the tile error after closing the floorplan overlay.
• Created an example in netjsongraph.js to replicate the tile error and pushed it on a branch.
• Started implementing the Figma design.

Tasks I will be working on next week:

• Address comments in any open PR

• Find a fix for the tile error after closing the floorplan overlay.
• Complete ui from Figma design.

Blockers:

• Bug: The echarts-leaflet library globally overrides the Leaflet CRS to L.CRS.Simple when rendering the indoor map. As a result, any interaction with the geographic map after rendering the indoor map uses the incorrect CRS, causing tile fetching errors.

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[July 14 - July 18]

List of Tasks I have worked on :

• [feature] Add API endpoint for indoor map coordinates #976 is ready for review

• [feature] Add floorplan map above dashboard map #680, addressed all comments
• [bug] Overlaying Indoor Map with L.CRS.Simple Overrides Geo Map Interaction and CRS Settings #397, Opened this PR
• [feature] Improve device popup UI and filtering #668,
  • Imlemented the figma design.
  • Updated the filtering logic (refer issue  #662 comment for the logic details)

Tasks I will be working on next week:

• Address comments in any open PR.
• Complete [feature] Improve device popup UI and filtering #668
•  Start working on on [feature] Make actions bookmarkable #238 and [feature] Add a URL in the device's change page to view the device on the full map #562

Blockers:

• Bug: The echarts-leaflet library globally overrides the Leaflet CRS to L.CRS.Simple when rendering the indoor map. As a result, any interaction with the geographic map after rendering the indoor map uses the incorrect CRS, causing tile fetching errors.

Regards,

Deepanshu Sahu

[Deepanshu Sahu]

[July 21 - July 25]

List of Tasks I have worked on :

• [feature] Add API endpoint for indoor map coordinates #976 is ready for review.

• [feature] Improve device popup UI and filtering #688 is ready for review.
  • Rebased the PR to show all changes in the device popup ui on geo map and floorplan overlay.
  • Fixed filtering logic for all edge cases.
  • Fixed css in device popup and floorplan overlay accoding to the Figma.
  • Added selenium tests.

Tasks I will be working on next week:

• Address comments in any open PR

• Start working on [feature] Make actions bookmarkable #238 and [feature] Add a URL in the device's change page to view the device on the full map #562

Blockers:

• Bug: The echarts-leaflet library globally overrides the Leaflet CRS to L.CRS.Simple when rendering the indoor map. As a result, any interaction with the geographic map after rendering the indoor map uses the incorrect CRS, causing tile fetching errors.

• Waiting for input from the ux team regarding the full-screen floor plan view and improving floor navigation on smaller screens.
• Waiting for the changes to be done on the geo map from the project Improve netjsongraph.js resiliency and visualization.

[Deepanshu Sahu]

[July 28 - August 01]

List of Tasks I have worked on :

• [feature] Add API endpoint for indoor map coordinates #976 is merged.
• Make ChannelsLiveServerTestCase compatible with Django 5.2 #2172 is merged.
• Worked on [feature] Improve device popup UI and filtering #688
  • Added an underlying div to allow any interaction with the geo map.
  • Make the floor navigation horizontal and below the floor plan container.
  • Fixed the image stretched issue discussed in the meeting.
• Explored solution for #402

Tasks I will be working on next week:

• Fix full issues coming in full-screen mode of the floorplan map
  • The node and labels are disappearing at a certain zoom level in full-screen mode.
  • The map appears to exit fullscreen visually, even though the fullscreen state is still active.
  • The floor navigation buttons become unresponsive.

• Start working on [feature] Make actions bookmarkable #238 and [feature] Add a URL in the device's change page to view the device on the full map #562

Blockers:

• Bug: The echarts-leaflet library globally overrides the Leaflet CRS to L.CRS.Simple when rendering the indoor map. As a result, any interaction with the geographic map after rendering the indoor map uses the incorrect CRS, causing tile fetching errors.

[Deepanshu Sahu]

[August 04 - August 08]

List of things I have worked on: 

• Had 3 days off from work.
• Explored solutions for rendering the floor navigation in full screen mode and, after discussing with Gagan able to make a fix for it, still need to refine it further and handle some edge cases.
• Stated working [feature] Make actions bookmarkable #238

Task I will work on next week:

• Make a fix for the below bug.
• Finalize the full Floorplan feature.
• Work on [feature] Make actions bookmarkable #238

Blockers:

• Bug: The echarts-leaflet library globally overrides the Leaflet CRS to L.CRS.Simple when rendering the indoor map. As a result, any interaction with the geographic map after rendering the indoor map uses the incorrect CRS, causing tile fetching errors.

[Deepanshu Sahu]

[August 11 - August15]

List of things I have worked on:

• Implemented the appraoch for full screen mode mention in the comment.
• Created a draft PR [feature] Update URL with node click of map interactions #471 (Work in progress).

Things I will work on next week:

• Fix the bug of two overlaying maps.
• Finalize [feature] Improve device popup UI and filtering #688
• Complete the implementation of [feature] Update URL with node click of map interactions #471

Blockers:

• Bug: The echarts-leaflet library globally overrides the Leaflet CRS to L.CRS.Simple when rendering the indoor map. As a result, any interaction with the geographic map after rendering the indoor map uses the incorrect CRS, causing tile fetching errors.

[Deepanshu Sahu]

[August 18 - August 22]

List of things I have worked on:

• Rebased [feature] Improve device popup UI and filtering #688 to build on top of changes by Yash.
• Pushed bounds in Params to get a specific view of the map in [feature] Update URL with node click of map interactions #471.
• Created a helper function to open the popup which will help in selenium tests and added a few more tests in #471.
• Made a fix finally for 2 overlaying maps by converting the coordinates to the provided crs on the fly.

Things I will work on next week:

•  Complete [feature] Update URL with node click of map interactions #471.
  • Fix a failing test in the ci.
  • Fix tooltip is not visible in the indoor map.
• Add fragments instead of params in [feature] Update URL with node click of map interactions #471.

Blockers:

• None

[Deepanshu Sahu]

[August 25 - August 29]

List of things I have worked on:

• Work on some previously pending PRs
  
  • [tests:fix] Switched to ChannelsLiveServerTestCase #465 is merged
  • [fix] Address field real-time update #171
• Updated the [feature] Improve device popup UI and filtering #688 and found out some issues mentioned in the comment

Things I will work on next week:

• Complete [feature] Improve device popup UI and filtering #688
• Implement the Solution discussed for the [feature] Update URL with node click of map interactions #471.

Blockers:

• None

[Deepanshu Sahu]

[September 1 - September 5]

List of things worked on:

• [feature] Improve device popup UI and filtering #688 is ready for review.
• [bug] Deep cloning of configuration by a custom utility function #438 is ready for review.
• [feature] Update URL with node click of map interactions #471 is ready for review.

Things I will work on next week:

• Complete [feature] Update URL with node click of map interactions #471 with tests and Docs.
• Implement mobile device feature.

Blockers:

• None

[Deepanshu Sahu]

[September 8 - September 12]

List of things worked on:

• [feature] Added indoor maps #688 is merged.

• [feature] Update URL with node click of map interactions #471 is ready for review.

• • Implemnted according to the comment.
  • Implemented state to be reserved when going back or farward in browser.
  • Completed the PR with tests and docs.
• [feature] Setting url fragments and applying to that data on both maps #703 is ready for review.

Things I will work on next week:

• Add selenium tests and docs for [feature] Setting url fragments and applying to that data on both maps #703
• Work on [feature] Allow maps to track mobile co-ordinates #563
  

Blockers:

• None