Uploaded image for project: 'Jalview'
  1. Jalview
  2. JAL-2438

Make findFeatureColour thread-safe and smarter

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.10.2
    • Component/s: Efficiency, sequencefeatures
    • Labels:
      None

      Description

      FeatureRenderer.findFeatureColour is called when determining feature colours for structures or the overview window. Because it is not currently thread-safe, 'clones' of FeatureRenderer are used for this. This is fragile (see JAL-2436 for a bug resulting).

      Proposed to explore how to make the method thread-safe so that no clones are needed, only a 'read-only' view of the FeatureRenderer.

      Performance is critical here so must be preserved - and improved where possible.

        Attachments

          Issue Links

          blocks

          Bug - A problem which impairs or prevents the functions of the product. JAL-2436 Structure menu "View | Colour By..." changes alignment colouring

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Task - A task that needs to be done. JAL-2443 Remove getResidueBoxColour from api.SequenceRenderer

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed
          related with

          Bug - A problem which impairs or prevents the functions of the product. JAL-2243 feature settings panel does not update as new features are added to alignment

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed

            Activity

              People

              Assignee:
              gmungoc Mungo Carstairs
              Reporter:
              gmungoc Mungo Carstairs
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: