Bug 86501 - Symbol skip option for curves
Summary: Symbol skip option for curves
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: plotting (show other bugs)
Version: 1.x
Platform: unspecified Solaris
: NOR wishlist
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-03 16:39 UTC by Nicolas Brisset
Modified: 2004-12-11 01:44 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Brisset 2004-08-03 16:39:14 UTC
Version:           0.99-devel (using KDE 3.2.1, compiled sources)
Compiler:          gcc version 3.3.2
OS:                SunOS (sun4u) release 5.8

Curves may optionally have symbols plotted along with the line. It would be good to add a "Skip" spinbox to the "Edit curve" dialog (right above the "width" spinbox for line width) and plot less points as the number increases: all points when the value is 0, one symbol every 2 points when the value is 1, etc...
Things would look better and I suppose it would save some memory/time in the repaints.
Comment 1 Netterfield 2004-08-03 19:27:45 UTC
-Skiping points is already done at the vector input level, with the option to boxcar filter first
-The line drawing is already smart, in that it only draws ~1 line per pixel (from max to min) if there are more than one point per pixel.
-When plotting individual data points rather than curves (which implies that every one is precious), it seems like a bad idea at best to randomly skip them.
Comment 2 Nicolas Brisset 2004-08-03 21:17:21 UTC
I still believe it would make sense. I actually often use that option to differentiate between various curves (especially when they are then plotted in black&white). Your suggestions do not fulfill my need:
- I do NOT want to skip points for the line itself, it's just that I don't want so many symbols.
- the line drawing may be smart, it does not help with the fact that too many symbols look bad and may even hide the curve partially
- when plotting individual data points, you can disable the lines and chosse NOT to skip points (which would be the default anyway)

Sorry, I fail to understand why you do not seem to want to implement this...
Comment 3 Nicolas Brisset 2004-10-19 14:05:59 UTC
I hope that the feature will be implemented now that my request has become clear in the light of explanations pertaining to bug #90196 (suggestions for nice black & white prints). Hopefully it is not too complicated...
Comment 4 Andrew Walker 2004-10-20 22:06:50 UTC
I would agree that this is a useful option. If the user wants the symbols simply to differntiate between different curves then they do not need every data point to have a symbol. 

Given that, the option should not be available when lines are disabled, and it would probably make more sense to make the symbol placement automated (based on the distance between plotted symbols).

The main attraction with this would be the performance increase with a large number of data points. This is currently slow - negating any performance gain derived from the smart line-drawing.
Comment 5 Andrew Walker 2004-12-11 01:44:30 UTC
CVS commit by arwalker: 

Added ability to define the density of points used to identify a line.
Disable and enable not only the edit fields, but also the labels associated
with items in the curve appearance widget.

CCMAIL: 86501-done@bugs.kde.org


  M +139 -82   curveappearancewidget.ui   1.21
  M +67 -10    curveappearancewidget.ui.h   1.18
  M +39 -10    kst2dplot.cpp   1.310
  M +15 -12    kstbasecurve.h   1.32
  M +43 -31    kstcurvedialog_i.cpp   1.74
  M +1 -0      kstfilterdialog_i.cpp   1.25
  M +1 -0      kstfitdialog_i.cpp   1.48
  M +3 -3      ksthsdialog_i.cpp   1.76
  M +2 -0      kstlinestyle.h   1.3
  M +1 -0      kstpsddialog_i.cpp   1.70
  M +7 -6      kstvcurve.cpp   1.53