pikers
/
piker
3
0
Fork 0
Code Issues 4 Pull Requests 6 Packages Projects Releases Wiki Activity

Always draw a last step line with px width=2

windows_testing_volume
Tyler Goodlet 2021-09-21 15:27:22 -04:00
parent c37e6e2440
commit f269fe9732
1 changed files with 42 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

62
piker/ui/_curve.py
View File

@ -102,14 +102,22 @@ class FastAppendCurve(pg.PlotCurveItem):
# we're basically only using the pen setting now... # we're basically only using the pen setting now...
super().__init__(*args, **kwargs) super().__init__(*args, **kwargs)
self._last_line: QLineF = None
self._xrange: tuple[int, int] = self.dataBounds(ax=0) self._xrange: tuple[int, int] = self.dataBounds(ax=0)
self._step_mode: bool = step_mode
self._fill = False
self.setBrush(hcolor(fill_color or color)) # all history of curve is drawn in single px thickness
self.setPen(hcolor(color)) self.setPen(hcolor(color))
# last segment is drawn in 2px thickness for emphasis
self.last_step_pen = pg.mkPen(hcolor(color), width=2)
self._last_line: QLineF = None
self._last_step_rect: QRectF = None
# flat-top style histogram-like discrete curve
self._step_mode: bool = step_mode
self._fill = False
self.setBrush(hcolor(fill_color or color))
# TODO: one question still remaining is if this makes trasform # TODO: one question still remaining is if this makes trasform
# interactions slower (such as zooming) and if so maybe if/when # interactions slower (such as zooming) and if so maybe if/when
# we implement a "history" mode for the view we disable this in # we implement a "history" mode for the view we disable this in
@ -192,8 +200,10 @@ class FastAppendCurve(pg.PlotCurveItem):
) )
path = self.path path = self.path
if self._step_mode:
# other merging ideas:
# https://stackoverflow.com/questions/8936225/how-to-merge-qpainterpaths
if self._step_mode:
if self._fill: if self._fill:
# XXX: super slow set "union" op # XXX: super slow set "union" op
self.path = self.path.united(append_path).simplified() self.path = self.path.united(append_path).simplified()
@ -222,11 +232,23 @@ class FastAppendCurve(pg.PlotCurveItem):
self.xData = x self.xData = x
self.yData = y self.yData = y
self._xrange = x[0], x[-1] x0, x_last = self._xrange = x[0], x[-1]
y_last = y[-1]
if self._step_mode: if self._step_mode:
self._last_step_rect = QRectF(x[-1] - 0.5, 0, x[-1] + 0.5, y[-1]) self._last_line = QLineF(
x_last - 0.5, 0,
x_last + 0.5, 0,
)
self._last_step_rect = QRectF(
x_last - 0.5, 0,
x_last + 0.5, y_last
)
else: else:
self._last_line = QLineF(x[-2], y[-2], x[-1], y[-1]) self._last_line = QLineF(
x[-2], y[-2],
x[-1], y_last
)
# trigger redraw of path # trigger redraw of path
# do update before reverting to cache mode # do update before reverting to cache mode
@ -277,8 +299,6 @@ class FastAppendCurve(pg.PlotCurveItem):
profiler = pg.debug.Profiler(disabled=not pg_profile_enabled()) profiler = pg.debug.Profiler(disabled=not pg_profile_enabled())
# p.setRenderHint(p.Antialiasing, True) # p.setRenderHint(p.Antialiasing, True)
p.setPen(self.opts['pen'])
if self._step_mode: if self._step_mode:
brush = self.opts['brush'] brush = self.opts['brush']
@ -286,17 +306,19 @@ class FastAppendCurve(pg.PlotCurveItem):
# p.drawRect(self._last_step_rect) # p.drawRect(self._last_step_rect)
p.fillRect(self._last_step_rect, brush) p.fillRect(self._last_step_rect, brush)
p.drawPath(self.path) # p.drawPath(self.path)
if self._fill: # profiler('.drawPath()')
print('FILLED')
p.fillPath(self.path, brush)
profiler('.drawPath()') # else:
p.setPen(self.last_step_pen)
p.drawLine(self._last_line)
profiler('.drawLine()')
else: p.setPen(self.opts['pen'])
p.drawLine(self._last_line) p.drawPath(self.path)
profiler('.drawLine()') profiler('.drawPath()')
p.drawPath(self.path) if self._fill:
profiler('.drawPath()') print('FILLED')
p.fillPath(self.path, brush)