| Re: [tracecompass-dev] Exporting CSV option |
|
Hello, Thanks for the update. PR #938 (https://github.com/eclipse-cdt-cloud/theia-trace-extension/pull/938)
has some overlap with your approach because it fires a signal to pass the csvString to the Theia backend, which triggers the FileDialogService to save the file. Once this feature gets merged you can use the export signal and the SaveFileDialog implementation
to extend export functionality to other views. From:
Hriday Panchasara <hriday.panchasara@xxxxxxxxxxxx> Hello, Thanks for the update. PR #938 (https://github.com/eclipse-cdt-cloud/theia-trace-extension/pull/938)
has some overlap with your approach because it fires a signal to pass the csvString to the Theia backend, which triggers the FileDialogService to save the file. Once this feature gets merged you can use the export signal and the SaveFileDialog implementation
to extend export functionality to other views. From:
Bernd Hufmann <bernd.hufmann@xxxxxxxxxxxx> Hi,
thanks for this suggestion. There is currently some effort ongoing to make the existing exportToCsv work when running in the vscode-trace-extension, because the current
implementation doesn't work in VsCode. See PR [1] and [2] for that. I see some overlap in this effort and this mailing post, but I haven't looked into the change in detail.
@Hriday Panchasara and @Alex could you both please coordinate the effort, either on mailing list or on the PRs? Thanks
a lot. Bernd From: tracecompass-dev <tracecompass-dev-bounces@xxxxxxxxxxx> on behalf of Alex Doan <vadoan@xxxxxxxxxxxxxx> Hello, I’m currently working on an “Export to csv” option that should work on all views, by adding a new state to
AbstractOutputComponent called exportCSV that can either be undefined or a string. Any view that wants to use this feature can set this state, following which a new option will appear in the drop-down menu. Clicking on this option will fire a
signal that export exportCSV string into a csv file and download it to the machine, similar to how it’s done in
https://github.com/eclipse-cdt-cloud/theia-trace-extension/pull/750. I also intend to change this pull request to use my export function if it’s implemented Please let me know if you have any comment, concern about this feature. Alex This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute
non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination,
distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful. |