Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 355084

Summary: [LTTng] latency analysis - the second trace to be selected sometimes doesn't appear
Product: z_Archived Reporter: Mathieu Denis <mathieu.denis>
Component: LinuxToolsAssignee: Bernd Hufmann <bernd.hufmann>
Status: CLOSED FIXED QA Contact: Francois Chouinard <fchouinard>
Severity: normal    
Priority: P3 CC: bernd.hufmann, INF4990.A2010, mathieu.denis
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:
Bug Depends on:    
Bug Blocks: 331467    
Attachments:
Description Flags
Clear the histogram selected timerange for the latency view bernd.hufmann: iplog+

Description Mathieu Denis CLA 2011-08-18 09:42:37 EDT 
Build Identifier: 20110526-1053

If you open up a first trace in the latency view and use at least one time the histogram to change the timerange, the second trace that you will select will not be displayed correctly or even not at all. The timerange is not clear upon the selection of a new trace.

Reproducible: Always

Steps to Reproduce:
1.Open a trace with the latency view and the histogram opened
2.Move/change the timerange of the histogram
3.Select a second trace
4.The latency view will probably be empty if the timerange you selected from the step2 doesn't overlaps the second trace timerange.
Comment 1 Mathieu Denis CLA 2011-08-18 13:43:39 EDT 
Created attachment 201733 [details]
Clear the histogram selected timerange for the latency view

Applicable to the package org.eclipse.linuxtools.lttng.ui
Comment 2 Mathieu Denis CLA 2011-08-18 13:45:12 EDT 
The problem has been fixed
Comment 3 Andrew Overholt CLA 2011-08-18 16:52:40 EDT 
Was the patch committed?  I don't see it here:

http://git.eclipse.org/c/linuxtools/org.eclipse.linuxtools.git/log/

If/when it is pushed, please mark the attachment with iplog+.  Thanks.
Comment 4 Mathieu Denis CLA 2011-08-18 16:59:51 EDT 
I don't have the permission to push!

Furthermore, the latency view hasn't been push, so this patch couldn't be applied.
Comment 5 Mathieu Denis CLA 2011-08-18 17:03:31 EDT 
We'll wait for Francois to approve this.
Comment 6 Andrew Overholt CLA 2011-08-19 16:05:17 EDT 
(In reply to comment #4)
> I don't have the permission to push!

Oh, "The problem has been fixed" in comment #2 confused me :)
Comment 7 Bernd Hufmann CLA 2012-02-14 09:05:38 EST 
Not an issue anymore after committing latest Latency Analysis feature implementation (see bug Bug 331467).
Comment 8 Francois Chouinard CLA 2012-03-26 14:04:33 EDT 
Delivered in 0.10.0