Draft PR that targets the work done towards TrackMate v9.

pom.xml

@@ -11,7 +11,7 @@
 
 	<groupId>sc.fiji</groupId>
 	<artifactId>TrackMate</artifactId>
-	<version>7.13.3-SNAPSHOT</version>
+	<version>8.0.0-SNAPSHOT</version>
 
 	<name>TrackMate</name>
 	<description>TrackMate plugin for Fiji.</description>
@@ -134,6 +134,7 @@
 	</ciManagement>
 
 	<properties>
+        <enforcer.skip>true</enforcer.skip>
 		<package-name>fiji.plugin.trackmate</package-name>
 		<license.licenseName>gpl_v3</license.licenseName>
 		<license.copyrightOwners>TrackMate developers.</license.copyrightOwners>
@@ -147,7 +148,8 @@
 
 		<fontchooser.version>2.5.2</fontchooser.version>
 		<javaGeom.version>0.11.1</javaGeom.version>
-		<imglib2.version>6.1.0</imglib2.version>
+		<imglib2-mesh.version>1.0.0</imglib2-mesh.version>
+		<bigvolumeviewer.version>0.3.1</bigvolumeviewer.version>
 	</properties>
 
 	<dependencies>
@@ -156,6 +158,16 @@
 			<groupId>sc.fiji</groupId>
 			<artifactId>fiji-lib</artifactId>
 		</dependency>
+		<dependency>
+			<groupId>sc.fiji</groupId>
+			<artifactId>labkit-ui</artifactId>
+			<exclusions>
+				<exclusion>
+					<groupId>net.imagej</groupId>
+					<artifactId>imagej-legacy</artifactId>
+				</exclusion>
+			</exclusions>
+		</dependency>
 
 		<!-- ImageJ dependencies -->
 		<dependency>
@@ -166,6 +178,38 @@
 			<groupId>net.imagej</groupId>
 			<artifactId>imagej-common</artifactId>
 		</dependency>
+		<dependency>
+			<groupId>net.imglib2</groupId>
+			<artifactId>imglib2-mesh</artifactId>
+			<version>${imglib2-mesh.version}</version>
+		</dependency>
+		<dependency>
+			<groupId>net.imagej</groupId>
+			<artifactId>imagej-ops</artifactId>
+		</dependency>
+		<dependency>
+			<groupId>sc.fiji</groupId>
+			<artifactId>bigvolumeviewer</artifactId>
+			<version>${bigvolumeviewer.version}</version>
+		</dependency>
+		<dependency>
+			<groupId>org.jogamp.jogl</groupId>
+			<artifactId>jogl-all</artifactId>
+		</dependency>
+		<dependency>
+			<groupId>org.jogamp.gluegen</groupId>
+			<artifactId>gluegen-rt</artifactId>
+		</dependency>			
+		<dependency>
+			<groupId>org.jogamp.gluegen</groupId>
+			<artifactId>gluegen-rt</artifactId>
+			<classifier>${scijava.natives.classifier.gluegen}</classifier>
+		</dependency>
+		<dependency>
+			<groupId>org.jogamp.jogl</groupId>
+			<artifactId>jogl-all</artifactId>
+			<classifier>${scijava.natives.classifier.jogl}</classifier>
+		</dependency>
 
 		<!-- ImgLib2 dependencies -->
 		<dependency>
@@ -202,16 +246,24 @@
 			<groupId>org.scijava</groupId>
 			<artifactId>scijava-listeners</artifactId>
 		</dependency>
+		<dependency>
+			<groupId>org.scijava</groupId>
+			<artifactId>ui-behaviour</artifactId>
+		</dependency>
 
 		<!-- Other dependencies -->
+		<dependency>
+			<groupId>com.google.guava</groupId>
+			<artifactId>guava</artifactId>
+		</dependency>		
 		<dependency>
 			<groupId>com.github.vlsi.mxgraph</groupId>
 			<artifactId>jgraphx</artifactId>
 		</dependency>
-		<dependency>
+		<!--<dependency>
 			<groupId>com.google.code.gson</groupId>
 			<artifactId>gson</artifactId>
-		</dependency>
+		</dependency> -->
 		<dependency>
 			<groupId>com.itextpdf</groupId>
 			<artifactId>itextpdf</artifactId>

src/main/java/fiji/plugin/trackmate/Dimension.java

@@ -31,8 +31,13 @@ public enum Dimension
 	POSITION,
 	VELOCITY,
 	LENGTH,
-	AREA, TIME, ANGLE, RATE, // count per frames
-	ANGLE_RATE, STRING; // for non-numeric features
+	AREA,
+	VOLUME,
+	TIME,
+	ANGLE,
+	RATE, // count per frames
+	ANGLE_RATE,
+	STRING; // for non-numeric features
 
 	/*
 	 * We separated length and position so that x,y,z are plotted on a different

src/main/java/fiji/plugin/trackmate/LoadTrackMatePlugIn.java

@@ -24,6 +24,7 @@
 import static fiji.plugin.trackmate.gui.Icons.TRACKMATE_ICON;
 
 import java.awt.Color;
+import java.awt.Dimension;
 import java.io.File;
 
 import javax.swing.JFrame;
@@ -65,8 +66,6 @@ public class LoadTrackMatePlugIn extends TrackMatePlugIn
 	@Override
 	public void run( final String filePath )
 	{
-//		GuiUtils.setSystemLookAndFeel();
-
 		final Logger logger = Logger.IJ_LOGGER;
 		File file;
 		if ( null == filePath || filePath.length() == 0 )
@@ -215,6 +214,8 @@ public void run( final String filePath )
 		frame.setIconImage( TRACKMATE_ICON.getImage() );
 		GuiUtils.positionWindow( frame, settings.imp.getWindow() );
 		frame.setVisible( true );
+		final Dimension size = frame.getSize();
+		frame.setSize( size.width, size.height + 1 );
 
 		// Text
 		final LogPanelDescriptor2 logDescriptor = ( LogPanelDescriptor2 ) sequence.logDescriptor();
@@ -230,7 +231,7 @@ public void run( final String filePath )
 		final String warning = reader.getErrorMessage();
 		if ( !warning.isEmpty() )
 		{
-			logger2.log( "Warnings occured during reading the file:\n"
+			logger2.log( "Warnings occurred during reading the file:\n"
 					+ "--------------------\n"
 					+ warning
 					+ "--------------------\n",
@@ -292,11 +293,12 @@ protected TmXmlReader createReader( final File lFile )
 
 	public static void main( final String[] args )
 	{
+		GuiUtils.setSystemLookAndFeel();
 		ImageJ.main( args );
 		final LoadTrackMatePlugIn plugIn = new LoadTrackMatePlugIn();
-		plugIn.run( null );
+//		plugIn.run( null );
 //		plugIn.run( "samples/FakeTracks.xml" );
-//		plugIn.run( "samples/MAX_Merged.xml" );
+		plugIn.run( "samples/MAX_Merged.xml" );
 //		plugIn.run( "c:/Users/tinevez/Development/TrackMateWS/TrackMate-Cellpose/samples/R2_multiC.xml" );
 //		plugIn.run( "/Users/tinevez/Desktop/230901_DeltaRcsB-ZipA-mCh_timestep5min_Stage9_reg/230901_DeltaRcsB-ZipA-mCh_timestep5min_Stage9_reg_merge65.xml" );
 	}

src/main/java/fiji/plugin/trackmate/Model.java

@@ -81,13 +81,13 @@ public class Model
 	 */
 	private int updateLevel = 0;
 
-	private final HashSet< Spot > spotsAdded = new HashSet< >();
+	private final HashSet< Spot > spotsAdded = new HashSet<>();
 
-	private final HashSet< Spot > spotsRemoved = new HashSet< >();
+	private final HashSet< Spot > spotsRemoved = new HashSet<>();
 
-	private final HashSet< Spot > spotsMoved = new HashSet< >();
+	private final HashSet< Spot > spotsMoved = new HashSet<>();
 
-	private final HashSet< Spot > spotsUpdated = new HashSet< >();
+	private final HashSet< Spot > spotsUpdated = new HashSet<>();
 
 	/**
 	 * The event cache. During a transaction, some modifications might trigger
@@ -96,15 +96,15 @@ public class Model
 	 * the event ID in this cache in the meantime. The event cache contains only
 	 * the int IDs of the events listed in {@link ModelChangeEvent}, namely
 	 * <ul>
-	 * <li> {@link ModelChangeEvent#SPOTS_COMPUTED}
-	 * <li> {@link ModelChangeEvent#TRACKS_COMPUTED}
-	 * <li> {@link ModelChangeEvent#TRACKS_VISIBILITY_CHANGED}
+	 * <li>{@link ModelChangeEvent#SPOTS_COMPUTED}
+	 * <li>{@link ModelChangeEvent#TRACKS_COMPUTED}
+	 * <li>{@link ModelChangeEvent#TRACKS_VISIBILITY_CHANGED}
 	 * </ul>
 	 * The {@link ModelChangeEvent#MODEL_MODIFIED} cannot be cached this way,
 	 * for it needs to be configured with modification spot and edge targets, so
 	 * it uses a different system (see {@link #flushUpdate()}).
 	 */
-	private final HashSet< Integer > eventCache = new HashSet< >();
+	private final HashSet< Integer > eventCache = new HashSet<>();
 
 	// OTHERS
 
@@ -120,7 +120,7 @@ public class Model
 	/**
 	 * The list of listeners listening to model content change.
 	 */
-	Set< ModelChangeListener > modelChangeListeners = new LinkedHashSet< >();
+	Set< ModelChangeListener > modelChangeListeners = new LinkedHashSet<>();
 
 	/*
 	 * CONSTRUCTOR
@@ -130,6 +130,7 @@ public Model()
 	{
 		featureModel = createFeatureModel();
 		trackModel = createTrackModel();
+		addModelChangeListener( new SpotMeshSliceCacheInvalidator() );
 	}
 
 	/*
@@ -154,7 +155,7 @@ protected TrackModel createTrackModel()
 	 * <p>
 	 * Subclassers can override this method to have the model work with their
 	 * own subclass of {@link FeatureModel}.
-	 * 
+	 *
 	 * @return a new instance of {@link FeatureModel}.
 	 */
 	protected FeatureModel createFeatureModel()
@@ -321,7 +322,7 @@ public void clearTracks( final boolean doNotify )
 
 	/**
 	 * Returns the {@link TrackModel} that manages the tracks for this model.
-	 * 
+	 *
 	 * @return the track model.
 	 */
 	public TrackModel getTrackModel()
@@ -447,7 +448,7 @@ public void notifyFeaturesComputed()
 	/**
 	 * Set the logger that will receive the messages from the processes
 	 * occurring within this trackmate.
-	 * 
+	 *
 	 * @param logger
 	 *            the {@link Logger} to use.
 	 */
@@ -458,7 +459,7 @@ public void setLogger( final Logger logger )
 
 	/**
 	 * Return the logger currently set for this model.
-	 * 
+	 *
 	 * @return the {@link Logger} used.
 	 */
 	public Logger getLogger()
@@ -544,7 +545,7 @@ public synchronized Spot moveSpotFrom( final Spot spotToMove, final Integer from
 	 * 	model.endUpdate();
 	 * }
 	 * </pre>
-	 * 
+	 *
 	 * @param spotToAdd
 	 *            the spot to add.
 	 * @param toFrame
@@ -593,8 +594,9 @@ public synchronized Spot removeSpot( final Spot spotToRemove )
 			if ( DEBUG )
 				System.out.println( "[TrackMateModel] Removing spot " + spotToRemove + " from frame " + fromFrame );
 
-			trackModel.removeSpot( spotToRemove ); 
-			// changes to edges will be caught automatically by the TrackGraphModel
+			trackModel.removeSpot( spotToRemove );
+			// changes to edges will be caught automatically by the
+			// TrackGraphModel
 			return spotToRemove;
 		}
 		if ( DEBUG )
@@ -626,7 +628,7 @@ public synchronized Spot removeSpot( final Spot spotToRemove )
 	public synchronized void updateFeatures( final Spot spotToUpdate )
 	{
 		spotsUpdated.add( spotToUpdate ); // Enlist for feature update when
-											// transaction is marked as finished
+		// transaction is marked as finished
 		final Set< DefaultWeightedEdge > touchingEdges = trackModel.edgesOf( spotToUpdate );
 		if ( null != touchingEdges )
 		{
@@ -766,7 +768,7 @@ public synchronized boolean setTrackVisibility( final Integer trackID, final boo
 	 * The copy is made of the same spot objects but on a different graph, that
 	 * can be safely edited. The copy does not include the feature values for
 	 * edges and tracks, but the features are declared.
-	 * 
+	 *
 	 * @return a new model.
 	 */
 	public Model copy()
@@ -810,7 +812,7 @@ public Model copy()
 				featureModel.getTrackFeatureShortNames(),
 				featureModel.getTrackFeatureDimensions(),
 				featureModel.getTrackFeatureIsInt() );
-		
+
 		// Feature values are not copied.
 		return copy;
 	}
@@ -841,7 +843,7 @@ private void flushUpdate()
 		final int nEdgesToSignal = trackModel.edgesAdded.size() + trackModel.edgesRemoved.size() + trackModel.edgesModified.size();
 
 		// Do we have tracks to update?
-		final HashSet< Integer > tracksToUpdate = new HashSet< >( trackModel.tracksUpdated );
+		final HashSet< Integer > tracksToUpdate = new HashSet<>( trackModel.tracksUpdated );
 
 		// We also want to update the tracks that have edges that were modified
 		for ( final DefaultWeightedEdge modifiedEdge : trackModel.edgesModified )
@@ -853,7 +855,7 @@ private void flushUpdate()
 		final int nSpotsToUpdate = spotsAdded.size() + spotsMoved.size() + spotsUpdated.size();
 		if ( nSpotsToUpdate > 0 )
 		{
-			final HashSet< Spot > spotsToUpdate = new HashSet< >( nSpotsToUpdate );
+			final HashSet< Spot > spotsToUpdate = new HashSet<>( nSpotsToUpdate );
 			spotsToUpdate.addAll( spotsAdded );
 			spotsToUpdate.addAll( spotsMoved );
 			spotsToUpdate.addAll( spotsUpdated );
@@ -958,4 +960,20 @@ private void flushUpdate()
 		}
 	}
 
+	private static class SpotMeshSliceCacheInvalidator implements ModelChangeListener
+	{
+
+		@Override
+		public void modelChanged( final ModelChangeEvent event )
+		{
+			if ( event.getEventID() != ModelChangeEvent.MODEL_MODIFIED )
+				return;
+
+			event.getSpots()
+					.stream()
+					.filter( s -> event.getSpotFlag( s ) == ModelChangeEvent.FLAG_SPOT_MODIFIED )
+					.filter( s -> ( s instanceof SpotMesh ) )
+					.forEach( s -> ( ( SpotMesh ) s ).resetZSliceCache() );
+		}
+	}
 }