Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-10830

Optimize the performance of multi-trace chart updates

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: SUIT
    • Labels:

      Description

      Currently, our multi-trace charts create a new plot on EVERY update. We need to to use Plotly's restyle and relayout to speed the updates like highlight change, changing style, adding/removing errors, etc.

      Ideally, we'd like to calculate and handle the changes in PlotlyWrapper. This is a very important issue to be resolved BEFORE we can start using multi-trace charts in tri-view.

        Attachments

          Issue Links

            Activity

            Hide
            loi Loi Ly added a comment -

            Implemented by applying only the updates where applicable. There is a slight increase on performance, but not enough when compared to HighChart.
            Also explored using WebGL(scattergl) as an option. Although draw/redraw performance was amazing, it's buggy and lack the features needed. This is something we'll revisit at a later time when plotly.js library is more robust.
            See PR for more details.

            Also fixed in this ticket:

            • update plotly.js to v1.28.2
            • use color-blind friendly colors when creating new trace
            • fixed FilterInfo.js fail when comparing sparsed array(rows).
            Show
            loi Loi Ly added a comment - Implemented by applying only the updates where applicable. There is a slight increase on performance, but not enough when compared to HighChart. Also explored using WebGL(scattergl) as an option. Although draw/redraw performance was amazing, it's buggy and lack the features needed. This is something we'll revisit at a later time when plotly.js library is more robust. See PR for more details. Also fixed in this ticket: update plotly.js to v1.28.2 use color-blind friendly colors when creating new trace fixed FilterInfo.js fail when comparing sparsed array(rows).

              People

              Assignee:
              loi Loi Ly
              Reporter:
              tatianag Tatiana Goldina
              Reviewers:
              Emmanuel Joliet, Tatiana Goldina
              Watchers:
              Emmanuel Joliet, Loi Ly, Tatiana Goldina, Xiuqin Wu [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: