examples icon indicating copy to clipboard operation
examples copied to clipboard

Updates to Streaming Timeseries

Open maximlt opened this issue 1 year ago • 2 comments

The example had to be merged quite quickly without having enough time to add a few improvements listed in the review:

For future extensions to this example (not this PR):

  • [ ] add event detection
  • [ ] add memory load in side plot for cpu stream
  • [ ] add image stream with pos estimation in additional side plot
  • [ ] add copy-pastable app at bottom

Originally posted by @droumis in https://github.com/holoviz-topics/examples/issues/469#issuecomment-2543047162

maximlt avatar Dec 16 '24 10:12 maximlt

There was also:

  • [ ] Get the main app to stream some data for a little while building the site, the objective being to run some more code so the notebook is more tested on the infrastructure, and possibly replace the existing screenshot by the real app. See https://github.com/holoviz-topics/examples/pull/469#discussion_r1880398349
  • The app has a notebook flag to declare whether it should be displayed in a template or not:
    • [ ] Isn't there a way to automatically detect the context? See https://github.com/holoviz-topics/examples/pull/469#discussion_r1880352906
    • [ ] The app uses two variables, layout and template, that it returns conditionally in __panel__(). In theory it could just have one (e.g. view) but apparently something odd was going on and needed more investigation, see https://github.com/holoviz-topics/examples/pull/469#discussion_r1880351849

maximlt avatar Dec 16 '24 10:12 maximlt

Also,

  • [ ] "Return to Stream Follow" needs to be investigated, as it's not working with the current implementation. It should be that "Zooming or panning temporarily pauses the viewport’s displayed range; clicking ‘reset’ restores the viewport range to follow the latest stream updates."
  • [ ] Use of pn.io.unlocked could use the some eyes from a senior engineer

droumis avatar Dec 16 '24 12:12 droumis