Skip to content

Commit e30f50f

Browse files
author
Stefan Hahmann
committed
Replace interface DataDisplayFrameSupplier by more reusable interface Plotable
1 parent b8b8759 commit e30f50f

8 files changed

Lines changed: 18 additions & 50 deletions

File tree

src/main/java/org/mastodon/mamut/views/grapher/DataDisplayFrameSupplier.java

Lines changed: 0 additions & 41 deletions
This file was deleted.

src/main/java/org/mastodon/mamut/views/grapher/GrapherGuiState.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
import org.mastodon.views.grapher.display.FeatureGraphConfig;
4242
import org.mastodon.views.grapher.display.FeatureSpecPair;
4343
import org.mastodon.views.grapher.display.GrapherSidePanel;
44+
import org.mastodon.views.grapher.display.Plotable;
4445
import org.mastodon.views.grapher.display.ScreenTransformState;
4546

4647
public class GrapherGuiState
@@ -118,10 +119,10 @@ private GrapherGuiState()
118119
loadGuiState( guiState, screenTransformState, sidePanel, defaultConfig, null );
119120
}
120121

121-
static < V extends Vertex< E > & HasTimepoint & HasLabel & Ref< V >, E extends Edge< V > & Ref< E > > void
122+
public static < V extends Vertex< E > & HasTimepoint & HasLabel & Ref< V >, E extends Edge< V > & Ref< E > > void
122123
loadGuiState( final Map< String, Object > guiState, final ScreenTransformState screenTransformState,
123124
final GrapherSidePanel sidePanel, final FeatureGraphConfig defaultConfig,
124-
final DataDisplayFrameSupplier< V, E > frameSupplier )
125+
final Plotable plotable )
125126
{
126127
// Read Screen Transform.
127128
final ScreenTransform screenTransform = ( ScreenTransform ) guiState.get( GRAPHER_TRANSFORM_KEY );
@@ -133,8 +134,8 @@ private GrapherGuiState()
133134
sidePanel.setGraphConfig( config );
134135

135136
// Plot with loaded transform and config.
136-
if ( frameSupplier != null )
137-
frameSupplier.getFrame().plot( screenTransform );
137+
if ( plotable != null )
138+
plotable.plot( screenTransform );
138139
}
139140

140141
private static < V extends Vertex< E > & HasTimepoint & HasLabel & Ref< V >, E extends Edge< V > & Ref< E > > void

src/main/java/org/mastodon/mamut/views/grapher/MamutBranchViewGrapher.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
import java.util.function.BiConsumer;
5858

5959
public class MamutBranchViewGrapher extends MamutBranchView< DataGraph< BranchSpot, BranchLink >, DataVertex, DataEdge >
60-
implements HasColoringModel, HasColorBarOverlay, DataDisplayFrameSupplier< BranchSpot, BranchLink >
60+
implements HasColoringModel, HasColorBarOverlay
6161
{
6262

6363
private final GrapherInitializer< BranchSpot, BranchLink > grapherInitializer;

src/main/java/org/mastodon/mamut/views/grapher/MamutBranchViewGrapherFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ public void restoreGuiState( final MamutBranchViewGrapher view, final Map< Strin
6666
super.restoreGuiState( view, guiState );
6767
DataDisplayFrame< BranchSpot, BranchLink > frame = view.getFrame();
6868
GrapherGuiState.loadGuiState( guiState, frame.getDataDisplayPanel().getScreenTransform(), frame.getVertexSidePanel(),
69-
MamutBranchViewGrapher.getFeatureGraphConfig(), view );
69+
MamutBranchViewGrapher.getFeatureGraphConfig(), view.getFrame() );
7070
}
7171

7272
@Override

src/main/java/org/mastodon/mamut/views/grapher/MamutViewGrapher.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@
5959
import net.imglib2.loops.LoopBuilder;
6060

6161
public class MamutViewGrapher extends MamutView< DataGraph< Spot, Link >, DataVertex, DataEdge >
62-
implements HasContextChooser< Spot >, HasColoringModel, HasColorBarOverlay, DataDisplayFrameSupplier< Spot, Link >
62+
implements HasContextChooser< Spot >, HasColoringModel, HasColorBarOverlay
6363
{
6464

6565
private final GrapherInitializer< Spot, Link > grapherInitializer;

src/main/java/org/mastodon/mamut/views/grapher/MamutViewGrapherFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ public void restoreGuiState( final MamutViewGrapher view, final Map< String, Obj
9898
super.restoreGuiState( view, guiState );
9999
DataDisplayFrame< Spot, Link > frame = view.getFrame();
100100
GrapherGuiState.loadGuiState( guiState, frame.getDataDisplayPanel().getScreenTransform(), frame.getVertexSidePanel(),
101-
MamutViewGrapher.getFeatureGraphConfig(), view );
101+
MamutViewGrapher.getFeatureGraphConfig(), view.getFrame() );
102102
}
103103

104104
@Override

src/main/java/org/mastodon/views/grapher/display/DataDisplayFrame.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@
6363
import org.mastodon.views.grapher.datagraph.ScreenTransform;
6464
import org.scijava.ui.behaviour.MouseAndKeyHandler;
6565

66-
public class DataDisplayFrame< V extends Vertex< E > & HasTimepoint & HasLabel, E extends Edge< V > > extends ViewFrame
66+
public class DataDisplayFrame< V extends Vertex< E > & HasTimepoint & HasLabel, E extends Edge< V > > extends ViewFrame implements Plotable
6767
{
6868
private static final long serialVersionUID = 1L;
6969

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
package org.mastodon.views.grapher.display;
2+
3+
import org.mastodon.views.grapher.datagraph.ScreenTransform;
4+
5+
public interface Plotable
6+
{
7+
void plot( final ScreenTransform screenTransform );
8+
}

0 commit comments

Comments
 (0)