HuskieRobotics / HuskieRobotics/SPOT

move CSV export to client-side to clarify analysis code

Open
#232 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
JavaScript
Stars
27
Forks
29
PR merge metrics
No merged PRs in 30d

Description

**Describe the bug**
The current analysis module is confusing as the analysis pipeline has to be able to run on both server-side for CSV exporting and client-side for analysis display. The code could be significantly cleaned up if the analysis pipeline was only run on the client side. I believe the only feature preventing this from occurring is exporting CSV files.

**Expected behavior**
[It appears](https://stackoverflow.com/questions/14964035/how-to-export-javascript-array-info-to-csv-on-client-side) that CSV files can be exported from the client-side. If this is the case, and if CSV export is the only feature that requires the analysis pipeline to run on the server side, then this code could be cleaned up and simplified. This could include using client-side modules for transformers instead of the current transformer template design.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.