added check for baseline stream in for live line plotting #164
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The only change I have made is to add an if statement which checks whether the stream that is to be plotted has name
baseline
if it does, no new plot is made.Motivation and Context
Aiming to solve problem where if a run is performed which contains baseline and primary (and maybe other plans) we don't really want to plot the baseline measurement. This can contain a lot of readings (=lines) that we don't really want to see in the plot
How Has This Been Tested?
I have only tested this by loading the widget using Jupyter and creating a run which has baseline and primary streams. I initially attempted to create a test in the pytests file but found that I couldn't use the
build_simple_run
function which was used in those tests because it only has one stream. TheRunBuilder()
would work, but I couldn't work out how to add incremenetingscan_id