Make points for Event DS plots show up

Close #263 (closed)

Make the default color for the points in dotForTime plots change with the background color. Currently, the points are colored white if the background is black and vice versa.

requested review from @ldam

assigned to @kienle

changed title from Fix the problem to Make points for Event DS plots show up

approved this merge request

unapproved this merge request

can you check in2017149.92EB.cf to see the number of points in Event DS2 are always double the actual points I can see.

They probably overlap. If you click on a point, you should see that two lines in the log are highlighted. My guess is that this is caused by the MR where you started reading both EH and ET packets. I don't see any code to check that only one of the two is read for an event.

Edit: We probably never caught this because Event DS points were not plotted before.

You are right. When I remove ET then the number is correct. Can you fix to just take one of the ET and EH if there are two of them.

or in one packet don't allow duplication.

If you’ve never seen ET without EH, I think we can just simply remove ET like before and comment that ET give same info as EH.

I will have to ask around on Monday to see if anyone has seen this situation before.

How do they know?

rt2ms reports if the EH packet is missing.

I can approve this MR and will write another MR for this problem.

approved this merge request

added 17 commits

• 634a00ad...ead31f32 - 16 commits from branch develop
• 6d65d96a - Fix the problem

Compare with previous version

enabled an automatic merge when the pipeline for 6d65d96a succeeds

merged

mentioned in commit c7ec594f