X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FOverviewCanvas.java;h=308764ba56b0c6b0fcb66fd5d491bbcc4dc7072f;hb=74d5ca6390288f6cd6cb445cccc728802806a29a;hp=5005f4e1d9005f0a4e14131082a116be5efec5a1;hpb=fb8c1854eae188200a5815dd097484cc6beaa434;p=jalview.git diff --git a/src/jalview/gui/OverviewCanvas.java b/src/jalview/gui/OverviewCanvas.java index 5005f4e..308764b 100644 --- a/src/jalview/gui/OverviewCanvas.java +++ b/src/jalview/gui/OverviewCanvas.java @@ -21,7 +21,10 @@ package jalview.gui; import jalview.api.AlignViewportI; -import jalview.renderer.seqfeatures.FeatureColourFinder; +import jalview.bin.Cache; +import jalview.datamodel.AlignmentI; +import jalview.renderer.OverviewRenderer; +import jalview.renderer.OverviewResColourFinder; import jalview.viewmodel.OverviewDimensions; import java.awt.Color; @@ -29,9 +32,10 @@ import java.awt.Dimension; import java.awt.Graphics; import java.awt.image.BufferedImage; -import javax.swing.JComponent; +import javax.swing.JPanel; -public class OverviewCanvas extends JComponent +@SuppressWarnings("serial") +public class OverviewCanvas extends JPanel { private static final Color TRANS_GREY = new Color(100, 100, 100, 25); @@ -41,30 +45,59 @@ public class OverviewCanvas extends JComponent private volatile boolean updaterunning = false; - private BufferedImage miniMe; + private boolean disposed = false; - private BufferedImage lastMiniMe = null; + BufferedImage lastMiniMe = null; // Can set different properties in this seqCanvas than // main visible SeqCanvas - private SequenceRenderer sr; + private final SequenceRenderer sr; - private jalview.renderer.seqfeatures.FeatureRenderer fr; + private final jalview.renderer.seqfeatures.FeatureRenderer fr; private OverviewDimensions od; - private AlignViewportI av; + private OverviewRenderer or = null; - public OverviewCanvas(OverviewDimensions overviewDims, - AlignViewportI alignvp) + private final AlignViewportI av; + + private final OverviewResColourFinder cf; + + private final ProgressPanel progressPanel; + + private boolean showSequenceFeatures; + + private boolean showAnnotation; + + private jalview.api.FeatureRenderer featureRenderer; + + private final OverviewPanel panel; + + private final boolean showProgress; + + public OverviewCanvas(OverviewPanel panel, + OverviewDimensions overviewDims, + AlignViewportI alignvp, ProgressPanel pp) { + this.panel = panel; od = overviewDims; + lastMiniMe = null; av = alignvp; - + progressPanel = pp; + showProgress = (pp != null); sr = new SequenceRenderer(av); sr.renderGaps = false; - sr.forOverview = true; fr = new jalview.renderer.seqfeatures.FeatureRenderer(av); + + boolean useLegacy = Cache.getDefault(Preferences.USE_LEGACY_GAP, false); + Color gapCol = Cache.getDefaultColour(Preferences.GAP_COLOUR, + jalview.renderer.OverviewResColourFinder.OVERVIEW_DEFAULT_GAP); + Color hiddenCol = Cache.getDefaultColour(Preferences.HIDDEN_COLOUR, + jalview.renderer.OverviewResColourFinder.OVERVIEW_DEFAULT_HIDDEN); + cf = new OverviewResColourFinder(useLegacy, gapCol, hiddenCol); + + setSize(od.getWidth(), od.getHeight()); + setPreferredSize(getSize()); // BH 2019.07.29 added } /** @@ -76,6 +109,7 @@ public class OverviewCanvas extends JComponent public void resetOviewDims(OverviewDimensions overviewDims) { od = overviewDims; + lastMiniMe = null; } /** @@ -88,16 +122,27 @@ public class OverviewCanvas extends JComponent { if (updaterunning) { - restart = true; + setRestart("restartDraw"); } else { updaterunning = true; + restart = false; } return restart; } } + private void setRestart(String why) + { + // System.out.println("OC restart true " + why); + restart = true; + if (or != null) + { + or.setRedraw(true); + } + } + /** * Draw the overview sequences * @@ -105,81 +150,172 @@ public class OverviewCanvas extends JComponent * true if sequence features are to be shown * @param showAnnotation * true if the annotation is to be shown - * @param transferRenderer + * @param featureRenderer * the renderer to transfer feature colouring from */ public void draw(boolean showSequenceFeatures, boolean showAnnotation, - FeatureRenderer transferRenderer) + jalview.api.FeatureRenderer featureRenderer) { - miniMe = null; - + this.showSequenceFeatures = showSequenceFeatures; + this.showAnnotation = showAnnotation; + this.featureRenderer = featureRenderer; if (showSequenceFeatures) { - fr.transferSettings(transferRenderer); + fr.transferSettings(featureRenderer); } - FeatureColourFinder finder = new FeatureColourFinder(fr); - - // why do we need to set preferred size again? was set in - // updateOverviewImage setPreferredSize(new Dimension(od.getWidth(), od.getHeight())); - - OverviewRenderer or = new OverviewRenderer(sr, finder, od); - miniMe = or.draw(od.getRows(av.getRanges(), av.getAlignment()), - od.getColumns(av.getRanges(), av.getColumnSelection())); - - Graphics mg = miniMe.getGraphics(); - - if (showAnnotation) + AlignmentI al = av.getAlignment(); + or = new OverviewRenderer(panel.ap, fr, od, al, av.getResidueShading(), + cf, showProgress); + if (showProgress) { - mg.translate(0, od.getSequencesHeight()); - or.drawGraph(mg, av.getAlignmentConservationAnnotation(), - av.getCharWidth(), od.getGraphHeight(), - od.getColumns(av.getRanges(), av.getColumnSelection())); - mg.translate(0, -od.getSequencesHeight()); + or.addPropertyChangeListener(progressPanel); } - System.gc(); + or.drawMiniMe(); + } + synchronized void finalizeDraw(BufferedImage miniMe) + { + if (showProgress && or != null) + { + or.removePropertyChangeListener(progressPanel); + } if (restart) { + or = null; restart = false; - draw(showSequenceFeatures, showAnnotation, transferRenderer); + if (!disposed) + { + draw(showSequenceFeatures, showAnnotation, featureRenderer); + } } else { + if (showAnnotation && or != null) + { + or.drawGraph(av.getAlignmentConservationAnnotation()); + } + or = null; updaterunning = false; lastMiniMe = miniMe; + repaint(); } } @Override public void paintComponent(Graphics g) { + + int w = getWidth(); + int h = getHeight(); + if (w == 0 || od.getBoxWidth() <= 0) + { + // BH 2019.07.27 removes two unnecessary paints, since boxwidth can be -1 + // or 0 during early-stage painting + return; + } + + boolean drawMe = (lastMiniMe != null); if (restart) { - if (lastMiniMe == null) + if (drawMe) { - g.setColor(Color.white); - g.fillRect(0, 0, getWidth(), getHeight()); + g.drawImage(lastMiniMe, 0, 0, w, h, this); } else { - g.drawImage(lastMiniMe, 0, 0, getWidth(), getHeight(), this); + g.setColor(Color.white); + g.fillRect(0, 0, w, h); } g.setColor(TRANS_GREY); - g.fillRect(0, 0, getWidth(), getHeight()); + g.fillRect(0, 0, w, h); + drawMe = false; } - else if (lastMiniMe != null) + else if (drawMe) { - g.drawImage(lastMiniMe, 0, 0, this); - if (lastMiniMe != miniMe) + // is this a resize? + if (w != od.getWidth() || h != od.getHeight()) { - g.setColor(TRANS_GREY); - g.fillRect(0, 0, getWidth(), getHeight()); + + lastMiniMe = null; + return; + // // if there is annotation, scale the alignment and annotation + // // separately + // if (od.getGraphHeight() <= 0 && od.getSequencesHeight() <= 0) + // { + // od.setWidth(w); + // od.setHeight(h); + // return; + // } + // try + // { + // BufferedImage topImage = lastMiniMe.getSubimage(0, 0, + // od.getWidth(), od.getSequencesHeight()); + // + // BufferedImage bottomImage = lastMiniMe.getSubimage(0, + // od.getSequencesHeight(), od.getWidth(), + // od.getGraphHeight()); + // + // // must be done at this point as we rely on using old width/height + // // above, and new width/height below + // od.setWidth(w); + // od.setHeight(h); + // + // // stick the images back together so lastMiniMe is consistent in the + // // event of a repaint - BUT probably not thread safe + // + // // right -- this fails with fast user action. + // + // lastMiniMe = new BufferedImage(w, h, BufferedImage.TYPE_INT_RGB); + // Graphics lg = lastMiniMe.getGraphics(); + // lg.drawImage(topImage, 0, 0, w, od.getSequencesHeight(), null); + // lg.drawImage(bottomImage, 0, od.getSequencesHeight(), w, + // od.getGraphHeight(), this); + // lg.dispose(); + // + // } catch (RasterFormatException e) + // { + // System.out.println("OC Raster Exception " + lastMiniMe.getWidth() + // + "/" + w + "," + lastMiniMe.getHeight() + "/" + h + " " + // + od.getSequencesHeight() + " " + od.getGraphHeight()); + // } + // BH 2019: removed -- this is now taken care of using vpbox in + // OverviewDimension + // // make sure the box is in the right place + // od.setBoxPosition(av.getAlignment().getHiddenSequences(), + // av.getAlignment().getHiddenColumns()); } } + if (drawMe) + { + g.drawImage(lastMiniMe, 0, 0, w, h, this); + } + // draw the box g.setColor(Color.red); + // System.out.println("OC paintComponent nd=" + ndraw + " nr=" + nrepaint + // + " np=" + ++npaint); od.drawBox(g); } + // private int ndraw, npaint, nrepaint; + + // @Override + // public void repaint() + // { + // System.out.println("OC repaint " + (++nrepaint)); + // super.repaint(); + // } + + public void dispose() + { + disposed = true; + od = null; + lastMiniMe = null; + synchronized (this) + { + setRestart("dispose"); + } + } + }