Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Migrate support of trace/experiment opening #7

Closed
bhufmann opened this issue Feb 16, 2021 · 0 comments · Fixed by #20
Closed

Migrate support of trace/experiment opening #7

bhufmann opened this issue Feb 16, 2021 · 0 comments · Fixed by #20
Assignees
@bhufmann
Labels
enhancement New feature or request

Comments

@bhufmann
Copy link
Collaborator

In the theia-trace-extension it is possible to select a directory or trace from the file explorer and execute "Open with -> Trace Viewer". After executing this the Trace Explorer is populated.

When no Trace is open, the "Open Trace" button is shown in the Trace Explorer and upon pressing the button the user can select a directory to find and open traces.

When executing that command, the command execution finds all CTF traces recursively in the directory and opens them in an experiment on the trace server. If the selection is a single file, then that file is opened as trace.
@bhufmann bhufmann added the enhancement New feature or request label Feb 16, 2021
@bhufmann bhufmann self-assigned this Mar 23, 2021
bhufmann added a commit to bhufmann/vscode-trace-extension that referenced this issue Mar 23, 2021
@bhufmann
Use react-components for open traces and views and related updates
4d96881 
- This change includes updates to propagate signals between webviews.
This will allow views to be updated when one view changes. For example,
when the user selects an open trace in the "Traces" view, the selection
is sent to the various react-components. The "Available Views" view
will use that signal as trigger to fetch the available views from
the remote server.

- Remove analysis and traces tree view from the extension. They are
replaced by the theia-trace-extension react components and
corresponding webviews.

- The classes AnalysisTree and TracesTree is still in the code base to
allow opening traces from the file system using the context sensitive
menu and "Open in Trace Viewer" command. This will have to be clean-up
in a later patch (+ using a cancelable progress monitor).

- The patch also includes a fix to make source-map work when debugging
the webview apps.

Needs eclipse-cdt-cloud/theia-trace-extension#295

Fixes eclipse-cdt-cloud#4
Fixes eclipse-cdt-cloud#7
Fixes eclipse-cdt-cloud#11
Fixes eclipse-cdt-cloud#16

Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
@bhufmann bhufmann mentioned this issue Mar 23, 2021
Merged
bhufmann added a commit to bhufmann/vscode-trace-extension that referenced this issue Mar 26, 2021
@bhufmann
Use react-components for open traces and views and related updates
1a2c825 
- This change includes updates to propagate signals between webviews.
This will allow views to be updated when one view changes. For example,
when the user selects an open trace in the "Traces" view, the selection
is sent to the various react-components. The "Available Views" view
will use that signal as trigger to fetch the available views from
the remote server.

- Remove analysis and traces tree view from the extension. They are
replaced by the theia-trace-extension react components and
corresponding webviews.

- The classes AnalysisTree and TracesTree is still in the code base to
allow opening traces from the file system using the context sensitive
menu and "Open in Trace Viewer" command. This will have to be clean-up
in a later patch (+ using a cancelable progress monitor).

- The patch also includes a fix to make source-map work when debugging
the webview apps.

Needs eclipse-cdt-cloud/theia-trace-extension#295

Fixes eclipse-cdt-cloud#4
Fixes eclipse-cdt-cloud#7
Fixes eclipse-cdt-cloud#11
Fixes eclipse-cdt-cloud#16

Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
bhufmann added a commit to bhufmann/vscode-trace-extension that referenced this issue Mar 31, 2021
@bhufmann
Use react-components for open traces and views and related updates
288d0ce 
- This change includes updates to propagate signals between webviews.
This will allow views to be updated when one view changes. For example,
when the user selects an open trace in the "Traces" view, the selection
is sent to the various react-components. The "Available Views" view
will use that signal as trigger to fetch the available views from
the remote server.

- Remove analysis and traces tree view from the extension. They are
replaced by the theia-trace-extension react components and
corresponding webviews.

- The classes AnalysisTree and TracesTree is still in the code base to
allow opening traces from the file system using the context sensitive
menu and "Open in Trace Viewer" command. This will have to be clean-up
in a later patch (+ using a cancelable progress monitor).

- The patch also includes a fix to make source-map work when debugging
the webview apps.

Needs eclipse-cdt-cloud/theia-trace-extension#295

Fixes eclipse-cdt-cloud#4
Fixes eclipse-cdt-cloud#7
Fixes eclipse-cdt-cloud#11
Fixes eclipse-cdt-cloud#16

Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
bhufmann added a commit to bhufmann/vscode-trace-extension that referenced this issue Mar 31, 2021
@bhufmann
Use react-components for open traces and views and related updates
37aebbe 
- This change includes updates to propagate signals between webviews.
This will allow views to be updated when one view changes. For example,
when the user selects an open trace in the "Traces" view, the selection
is sent to the various react-components. The "Available Views" view
will use that signal as trigger to fetch the available views from
the remote server.

- Remove analysis and traces tree view from the extension. They are
replaced by the theia-trace-extension react components and
corresponding webviews.

- The classes AnalysisTree and TracesTree is still in the code base to
allow opening traces from the file system using the context sensitive
menu and "Open in Trace Viewer" command. This will have to be clean-up
in a later patch (+ using a cancelable progress monitor).

- The patch also includes a fix to make source-map work when debugging
the webview apps.

Needs eclipse-cdt-cloud/theia-trace-extension#295

Fixes eclipse-cdt-cloud#7
Fixes eclipse-cdt-cloud#11
Fixes eclipse-cdt-cloud#16

Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
bhufmann added a commit to bhufmann/vscode-trace-extension that referenced this issue Mar 31, 2021
@bhufmann
Use react-components for open traces and views and related updates
f086fff 
- This change includes updates to propagate signals between webviews.
This will allow views to be updated when one view changes. For example,
when the user selects an open trace in the "Traces" view, the selection
is sent to the various react-components. The "Available Views" view
will use that signal as trigger to fetch the available views from
the remote server.

- Remove analysis and traces tree view from the extension. They are
replaced by the theia-trace-extension react components and
corresponding webviews.

- The classes AnalysisTree and TracesTree is still in the code base to
allow opening traces from the file system using the context sensitive
menu and "Open in Trace Viewer" command. This will have to be clean-up
in a later patch (+ using a cancelable progress monitor).

- The patch also includes a fix to make source-map work when debugging
the webview apps.

Needs eclipse-cdt-cloud/theia-trace-extension#295

Fixes eclipse-cdt-cloud#7
Fixes eclipse-cdt-cloud#11
Fixes eclipse-cdt-cloud#16

Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
bhufmann added a commit that referenced this issue Apr 5, 2021
@bhufmann
Use react-components for open traces and views and related updates
1287ba2 
- This change includes updates to propagate signals between webviews.
This will allow views to be updated when one view changes. For example,
when the user selects an open trace in the "Traces" view, the selection
is sent to the various react-components. The "Available Views" view
will use that signal as trigger to fetch the available views from
the remote server.

- Remove analysis and traces tree view from the extension. They are
replaced by the theia-trace-extension react components and
corresponding webviews.

- The classes AnalysisTree and TracesTree is still in the code base to
allow opening traces from the file system using the context sensitive
menu and "Open in Trace Viewer" command. This will have to be clean-up
in a later patch (+ using a cancelable progress monitor).

- The patch also includes a fix to make source-map work when debugging
the webview apps.

Needs eclipse-cdt-cloud/theia-trace-extension#295

Fixes #7
Fixes #11
Fixes #16

Signed-off-by: Bernd Hufmann <Bernd.Hufmann@ericsson.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@bhufmann bhufmann

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use react-components for open traces and views and related updates bhufmann/vscode-trace-extension
1 participant
@bhufmann