jankae / LibreVNA

100kHz to 6GHz 2 port USB based VNA
GNU General Public License v3.0
1.08k stars 204 forks source link

Add keyboard shortcut for CSV export #253

Open NewMKE opened 3 months ago

NewMKE commented 3 months ago

Summary

I would like to propose an enhancement to the graphic user interface (GUI) that allows users to export data in CSV format using a keyboard shortcut. Specifically, I'm suggesting a combination such as Control + S or another combination of your choice.

The motivation behind this feature is straightforward: shortcuts can significantly improve efficiency by reducing the number of clicks required to perform a common task. So, having a shortcut for exporting data in CSV format would save time and effort.

More importantly, I'm interested in automating data export processes using LabVIEW, which requires saving data to CSV files. Having a dedicated keyboard shortcut for CSV export would benefit this process by making it more convenient to integrate with my workflows.

I believe that adding a keyboard shortcut for CSV export would be a valuable addition to the GUI. Please let me know if my request is reasonable and let me know if this is possible!

Paths to implement it

Screenshot 2024-05-21 215210 As shown in the picture, a shortcut for the export button would largely benefit my work.

Extra information

It would also be helpful if you let me know which part of the code for the GUI is related to the shortcut adding! If you cannot focus on this task currently because of your schedule, I can, hopefully, work on it by myself and give you an update. Since I am not an expert in C/C++, so I would like to seek help from you, including where part should I add the code to! Thank you in advance and I am looking forward to hearing back from you!

jankae commented 3 months ago

Hi, thanks for the suggestion and offer to help :) The problem is that there are no keyboard shortcuts at all in the GUI so far. This could be a feature that I implement in the future but so far I have no concrete plans for it, sorry.

If you are just trying to export data automatically, take a look at the SCPI server API. It supports reading trace data and also has a rudimentary direct export to Touchstone (see #101 and #108 for details). No direct export to CSV though.