Le 21/08/2012 22:18, Philippe Mouawad a ecrit :
Hello Milamber,
Profiling a little RespTimeGraphVisualizer, I think Pattern used for Sample
Label Selection regexp should be created only once instead of doing so on
each add.
Profiling, it takes 10% of time for loading a CSV file.
Maybe best place would be when reload data is clicked.
Yes you're probably right. I will make the change.
Writing this mail, I also think something in ergonomy might be not clear:
- For example, as a user I ask myself (without looking at code) does
"Display Graph" take into account Sampler label selection or must I click
"Reload Data" ?
You must click on Reload data (to reload the data with the filter
applied). Perhaps, the button label must be "Apply filter".
- When switching to Graph tab, is it equivalent to Display Graph or not ?
Yes. I can add a event listener when we switch to Graph tab to execute
the Display graph action.
- Same for "Apply interval"
Maybe there should be only one button called "Apply Changes" and switching
to tab would take all changes into account.
I don't believe. The Apply interval and Reload data action have a
internal process which may take few seconds when there are a lot of
results. The other settings are immediately applied on graph (no wait time).
If we have the internal process reload data at each setting changes, the
listener will be slow to display the graph every time with a significant
number of results.
