Merge remote-tracking branch 'origin/develop' into Story80_BoatCustomization

# Conflicts:
#	src/main/java/seng302/gameServer/GameState.java
#	src/main/java/seng302/gameServer/ServerPacketParser.java
#	src/main/java/seng302/gameServer/ServerToClientThread.java
#	src/main/java/seng302/gameServer/messages/ChatterMessage.java
#	src/main/java/seng302/gameServer/messages/MarkRoundingMessage.java
#	src/main/java/seng302/model/ServerYacht.java
#	src/main/java/seng302/visualiser/ClientToServerThread.java
#	src/main/java/seng302/visualiser/GameClient.java
#	src/main/java/seng302/visualiser/GameView.java

src/main/java/seng302/gameServer/GameState.java

@@ -104,7 +104,6 @@ public class GameState implements Runnable {
 
         resetStartTime();
 
-        new Thread(this).start();   //Run the auto updates on the game state
         new Thread(this, "GameState").start();   //Run the auto updates on the game state
 
         marks = new MarkOrder().getAllMarks();
@@ -186,6 +185,14 @@ public class GameState implements Runnable {
         return windDirection;
     }
 
+    public static void setWindDirection(Double newWindDirection) {
+        windDirection = newWindDirection;
+    }
+
+    public static void setWindSpeed(Double newWindSpeed) {
+        windSpeed = newWindSpeed;
+    }
+
     public static Double getWindSpeedMMS() {
         return windSpeed;
     }
@@ -256,7 +263,6 @@ public class GameState implements Runnable {
         }
     }
 
-
     /**
      * Called periodically in this GameState thread to update the GameState values
      */
@@ -277,6 +283,8 @@ public class GameState implements Runnable {
                 checkForLegProgression(yacht);
                 raceFinished = false;
             }
+
+
         }
 
         if (raceFinished) {
@@ -335,17 +343,20 @@ public class GameState implements Runnable {
                 notifyMessageListeners(
                     new YachtEventCodeMessage(serverYacht.getSourceId())
                 );
-            } else if (checkBoundaryCollision(serverYacht)) {
+            }
-                serverYacht.setLocation(
+            else{
-                    calculateBounceBack(serverYacht, serverYacht.getLocation(),
+                if (checkBoundaryCollision(serverYacht)) {
-                        BOUNCE_DISTANCE_YACHT)
+                    serverYacht.setLocation(
-                );
+                            calculateBounceBack(serverYacht, serverYacht.getLocation(),
-                serverYacht.setCurrentVelocity(
+                                    BOUNCE_DISTANCE_YACHT)
-                    serverYacht.getCurrentVelocity() * COLLISION_VELOCITY_PENALTY
+                    );
-                );
+                    serverYacht.setCurrentVelocity(
-                notifyMessageListeners(
+                            serverYacht.getCurrentVelocity() * COLLISION_VELOCITY_PENALTY
-                    new YachtEventCodeMessage(serverYacht.getSourceId())
+                    );
-                );
+                    notifyMessageListeners(
+                            new YachtEventCodeMessage(serverYacht.getSourceId())
+                    );
+                }
             }
         }
     }
@@ -409,7 +420,7 @@ public class GameState implements Runnable {
     }
 
 
-    /**
+    /**        lobbyController.setPlayerListSource(clientLobbyList);
      * 4 Different cases of progression in the race 1 - Passing the start line 2 - Passing any
      * in-race Gate 3 - Passing any in-race Mark 4 - Passing the finish line
      *
@@ -432,6 +443,7 @@ public class GameState implements Runnable {
         }
 
         if (hasProgressed) {
+            yacht.incrementLegNumber();
             sendMarkRoundingMessage(yacht);
             logMarkRounding(yacht);
             yacht.setHasPassedLine(false);
@@ -654,7 +666,7 @@ public class GameState implements Runnable {
         // TODO: 13/8/17 figure out the rounding side, rounded mark source ID and boat status.
         Message markRoundingMessage = new MarkRoundingMessage(0, 0,
             sourceID, RoundingBoatStatus.RACING, roundingMark.getRoundingSide(), markType,
-            roundingMark.getSourceID());
+            currentMarkSeqID + 1);
 
         notifyMessageListeners(markRoundingMessage);
     }

src/main/java/seng302/gameServer/HeartbeatThread.java

@@ -13,7 +13,7 @@ import seng302.gameServer.messages.Message;
  * Will call .clientDisconnected on the delegate when a heartbeat message
  * cannot be sent to a player
  */
-public class HeartbeatThread extends Thread{
+public class HeartbeatThread implements Runnable {
     private final int HEARTBEAT_PERIOD = 200;
     private ClientConnectionDelegate delegate;
     private Integer seqNum;
@@ -23,6 +23,9 @@ public class HeartbeatThread extends Thread{
         this.delegate =  delegate;
         seqNum = 0;
         disconnectedPlayers = new Stack<>();
+
+        Thread thread = new Thread(this, "HeartBeat");
+        thread.start();
     }
 
     /**

src/main/java/seng302/gameServer/MainServerThread.java

@@ -5,6 +5,7 @@ import java.net.ServerSocket;
 import java.time.LocalDateTime;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Random;
 import java.util.Timer;
 import java.util.TimerTask;
 import seng302.gameServer.messages.BoatSubMessage;
@@ -20,7 +21,6 @@ import seng302.model.PolarTable;
 import seng302.model.ServerYacht;
 import seng302.model.mark.CompoundMark;
 import seng302.utilities.GeoUtility;
-import seng302.visualiser.GameClient;
 
 /**
  * A class describing the overall server, which creates and collects server threads for each client
@@ -29,12 +29,17 @@ import seng302.visualiser.GameClient;
 public class MainServerThread implements Runnable, ClientConnectionDelegate {
 
     private static final int PORT = 4942;
-    private static final Integer CLIENT_UPDATES_PER_SECOND = 10;
+    private static final Integer CLIENT_UPDATES_PER_SECOND = 60;
     private static final int LOG_LEVEL = 1;
     private static final int WARNING_TIME = 10 * -1000;
     private static final int PREPATORY_TIME = 5 * -1000;
     public static final int TIME_TILL_START = 10 * 1000;
 
+    private static final int MAX_WIND_SPEED = 12000;
+    private static final int MIN_WIND_SPEED = 8000;
+
+    public static int windSpeed = 1000;
+
     private boolean terminated;
 
     private Thread thread;
@@ -42,8 +47,6 @@ public class MainServerThread implements Runnable, ClientConnectionDelegate {
     private ServerSocket serverSocket = null;
     private ArrayList<ServerToClientThread> serverToClientThreads = new ArrayList<>();
 
-    private GameClient gameClient;
-
     public MainServerThread() {
         new GameState("localhost");
         try {
@@ -55,19 +58,15 @@ public class MainServerThread implements Runnable, ClientConnectionDelegate {
         GameState.addMarkPassListener(this::broadcastMessage);
         terminated = false;
         thread = new Thread(this, "MainServer");
+        startUpdatingWind();
         thread.start();
     }
 
 
     public void run() {
-        ServerListenThread serverListenThread;
-        HeartbeatThread heartbeatThread;
 
-        serverListenThread = new ServerListenThread(serverSocket, this);
+        new HeartbeatThread(this);
-        heartbeatThread = new HeartbeatThread(this);
+        new ServerListenThread(serverSocket, this);
-
-        heartbeatThread.start();
-        serverListenThread.start();
 
         //You should handle interrupts in some way, so that the thread won't keep on forever if you exit the app.
         while (!terminated) {
@@ -121,6 +120,45 @@ public class MainServerThread implements Runnable, ClientConnectionDelegate {
         }
     }
 
+    private static void updateWind(){
+        Integer direction = GameState.getWindDirection().intValue();
+        Integer windSpeed = GameState.getWindSpeedMMS().intValue();
+
+        Random random = new Random();
+
+        if (Math.floorMod(random.nextInt(), 2) == 0){
+            direction += random.nextInt(4);
+            windSpeed += random.nextInt(100) + 500;
+        }
+        else{
+            direction -= random.nextInt(4);
+            windSpeed -= random.nextInt(100) + 500;
+        }
+
+        direction = Math.floorMod(direction, 360);
+
+        if (windSpeed > MAX_WIND_SPEED){
+            windSpeed -= random.nextInt(1000);
+        }
+
+        if (windSpeed <= MIN_WIND_SPEED){
+            windSpeed += random.nextInt(1000);
+        }
+
+        GameState.setWindSpeed(Double.valueOf(windSpeed));
+        GameState.setWindDirection(direction.doubleValue());
+    }
+
+    private static void startUpdatingWind(){
+        Timer timer = new Timer();
+        timer.schedule(new TimerTask() {
+            @Override
+            public void run() {
+                updateWind();
+            }
+        }, 0, 500);
+    }
+
 
     static void serverLog(String message, int logLevel) {
         if (logLevel <= LOG_LEVEL) {
@@ -202,7 +240,8 @@ public class MainServerThread implements Runnable, ClientConnectionDelegate {
 
         for (Player player : GameState.getPlayers()) {
             ServerYacht y = player.getYacht();
-            BoatSubMessage m = new BoatSubMessage(y.getSourceId(), y.getBoatStatus(), 0,
+            BoatSubMessage m = new BoatSubMessage(y.getSourceId(), y.getBoatStatus(),
+                y.getLegNumber(),
                 0, 0, 1234L,
                 1234L);
             boatSubMessages.add(m);
@@ -236,25 +275,17 @@ public class MainServerThread implements Runnable, ClientConnectionDelegate {
         terminated = true;
     }
 
-    /**
-     * Pass GameClient to main server thread so it can access the properties inside.
-     *
-     * @param gameClient gameClient
-     */
-    public void setGameClient(GameClient gameClient) {
-        this.gameClient = gameClient;
-    }
-
     /**
      * Initialise boats to specific spaced out geopoints behind starting line.
      */
     private void initialiseBoatPositions() {
         // Getting the start line compound marks
-        CompoundMark cm = gameClient.getCourseData().getCompoundMarks().get(1);
+//        if (gameClient== null) {
-        GeoPoint startMark1 = new GeoPoint(cm.getMarks().get(0).getLat(),
+//            return;
-            cm.getMarks().get(0).getLng());
+//        }
-        GeoPoint startMark2 = new GeoPoint(cm.getMarks().get(1).getLat(),
+        CompoundMark cm = GameState.getMarkOrder().getMarkOrder().get(0);
-            cm.getMarks().get(1).getLng());
+        GeoPoint startMark1 = cm.getSubMark(1);
+        GeoPoint startMark2 = cm.getSubMark(2);
 
         // Calculating midpoint
         Double perpendicularAngle = GeoUtility.getBearing(startMark1, startMark2);

src/main/java/seng302/gameServer/ServerListenThread.java

@@ -8,13 +8,16 @@ import java.net.Socket;
  * A class for a thread to listen to connections
  * Created by wmu16 on 11/07/17.
  */
-public class ServerListenThread extends Thread{
+public class ServerListenThread implements Runnable {
     private ServerSocket serverSocket;
     private ClientConnectionDelegate delegate;
 
     public ServerListenThread(ServerSocket serverSocket, ClientConnectionDelegate delegate){
         this.serverSocket = serverSocket;
         this.delegate = delegate;
+
+        Thread thread = new Thread(this, "ServerListen");
+        thread.start();
     }
 
     /**

src/main/java/seng302/gameServer/ServerToClientThread.java

@@ -30,6 +30,7 @@ import seng302.gameServer.messages.RegistrationResponseStatus;
 import seng302.gameServer.messages.XMLMessage;
 import seng302.gameServer.messages.XMLMessageSubType;
 import seng302.gameServer.messages.YachtEventCodeMessage;
+import seng302.gameServer.messages.YachtEventCodeMessage;
 import seng302.model.Player;
 import seng302.model.ServerYacht;
 import seng302.model.stream.packets.PacketType;
@@ -37,6 +38,19 @@ import seng302.model.stream.packets.StreamPacket;
 import seng302.model.stream.xml.generator.Race;
 import seng302.model.stream.xml.generator.Regatta;
 import seng302.utilities.XMLGenerator;
+import seng302.gameServer.messages.BoatAction;
+import seng302.gameServer.messages.BoatLocationMessage;
+import seng302.gameServer.messages.BoatSubMessage;
+import seng302.gameServer.messages.ClientType;
+import seng302.gameServer.messages.Message;
+import seng302.gameServer.messages.RaceStatus;
+import seng302.gameServer.messages.RaceStatusMessage;
+import seng302.gameServer.messages.RaceType;
+import seng302.gameServer.messages.RegistrationResponseMessage;
+import seng302.gameServer.messages.RegistrationResponseStatus;
+import seng302.gameServer.messages.XMLMessage;
+import seng302.gameServer.messages.XMLMessageSubType;
+import seng302.model.ServerYacht;
 
 /**
  * A class describing a single connection to a Client for the purposes of sending and receiving on
@@ -186,6 +200,7 @@ public class ServerToClientThread implements Runnable, Observer {
                     long computedCrc = checksum.getValue();
                     long packetCrc = Message.bytesToLong(getBytes(4));
                     if (computedCrc == packetCrc) {
+                        //System.out.println("RECEIVED A PACKET");
                         switch (PacketType.assignPacketType(type, payload)) {
                             case BOAT_ACTION:
                                 BoatAction actionType = ServerPacketParser

src/main/java/seng302/gameServer/messages/ChatterMessage.java

@@ -1,6 +1,8 @@
 package seng302.gameServer.messages;
 
-
+/**
+ * Created by kre39 on 20/07/17.
+ */
 public class ChatterMessage extends Message {
 
     private final long MESSAGE_VERSION_NUMBER = 1;

src/main/java/seng302/gameServer/messages/MarkRoundingMessage.java

@@ -0,0 +1,58 @@
+package seng302.gameServer.messages;
+
+public class MarkRoundingMessage extends Message{
+    private final long MESSAGE_VERSION_NUMBER = 1;
+    private final int MESSAGE_SIZE = 21;
+
+    private long time;
+    private long ackNumber;
+    private long raceId;
+    private long sourceId;
+    private RoundingBoatStatus boatStatus;
+    private RoundingSide roundingSide;
+    private long markId;
+
+
+    /**
+     * This message is sent when a boat passes a mark, start line, or finish line
+     * The purpose of this is to record the time when yachts cross marks
+     * @param ackNumber ackNumber
+     * @param raceId raceId
+     * @param sourceId boatSourceId
+     * @param roundingBoatStatus roundingBoatStatus
+     * @param roundingSide roundingSide
+     * @param markId markId
+     */
+    public MarkRoundingMessage(int ackNumber, int raceId, int sourceId, RoundingBoatStatus roundingBoatStatus,
+        RoundingSide roundingSide, MarkType markType, int markId) {
+        this.time = System.currentTimeMillis();
+        this.ackNumber = ackNumber;
+        this.raceId = raceId;
+        this.sourceId = sourceId;
+        this.boatStatus = roundingBoatStatus;
+        this.roundingSide = roundingSide;
+        this.markId = markId;
+
+        setHeader(new Header(MessageType.MARK_ROUNDING, 1, (short) getSize()));
+        allocateBuffer();
+        writeHeaderToBuffer();
+
+        putByte((byte) MESSAGE_VERSION_NUMBER);
+        putInt((int) time, 6);
+        putInt((int) ackNumber, 2);
+        putInt((int) raceId, 4);
+        putInt((int) sourceId, 4);
+        putByte((byte) boatStatus.getCode());
+        putByte((byte) roundingSide.getCode());
+        putByte((byte) markType.getCode());
+        putByte((byte) markId);
+
+        writeCRC();
+        rewind();
+    }
+
+    @Override
+    public int getSize() {
+        return MESSAGE_SIZE;
+    }
+}

src/main/java/seng302/gameServer/messages/RaceStatusMessage.java

@@ -39,7 +39,7 @@ public class RaceStatusMessage extends Message{
         this.raceId = raceId;
         this.raceStatus = raceStatus;
         this.expectedStartTime = expectedStartTime;
-        this.raceWindDirection = raceWindDirection * windDirFactor;
+        this.raceWindDirection = raceWindDirection * windDirFactor+100.0;
         this.windSpeed = windSpeed;
         this.numBoatsInRace = numBoatsInRace;
         this.raceType = raceType;

src/main/java/seng302/model/ClientYacht.java

@@ -8,6 +8,8 @@ import java.util.Observable;
 import java.util.Observer;
 import javafx.beans.property.ReadOnlyDoubleProperty;
 import javafx.beans.property.ReadOnlyDoubleWrapper;
+import javafx.beans.property.ReadOnlyIntegerProperty;
+import javafx.beans.property.ReadOnlyIntegerWrapper;
 import javafx.beans.property.ReadOnlyLongProperty;
 import javafx.beans.property.ReadOnlyLongWrapper;
 import javafx.scene.paint.Color;
@@ -28,19 +30,24 @@ public class ClientYacht extends Observable {
             Boolean sailsIn, double velocity);
     }
 
+    @FunctionalInterface
+    public interface MarkRoundingListener {
+        void notifyRounding(ClientYacht yacht, CompoundMark markPassed, int legNumber);
+    }
+
     private Logger logger = LoggerFactory.getLogger(ClientYacht.class);
 
 
-    //BOTH AFAIK
     private String boatType;
     private Integer sourceId;
     private String hullID; //matches HullNum in the XML spec.
     private String shortName;
     private String boatName;
     private String country;
+    private Integer position;
 
     private Long estimateTimeAtFinish;
-    private Boolean sailIn = false;
+    private Boolean sailIn = true;
     private Integer currentMarkSeqID = 0;
     private Long markRoundTime;
     private Long timeTillNext;
@@ -50,13 +57,13 @@ public class ClientYacht extends Observable {
     private Integer boatStatus;
     private Double currentVelocity;
 
-    //CLIENT SIDE
     private List<YachtLocationListener> locationListeners = new ArrayList<>();
+    private List<MarkRoundingListener> markRoundingListeners = new ArrayList<>();
     private ReadOnlyDoubleWrapper velocityProperty = new ReadOnlyDoubleWrapper();
     private ReadOnlyLongWrapper timeTillNextProperty = new ReadOnlyLongWrapper();
     private ReadOnlyLongWrapper timeSinceLastMarkProperty = new ReadOnlyLongWrapper();
+    private ReadOnlyIntegerWrapper placingProperty = new ReadOnlyIntegerWrapper();
     private CompoundMark lastMarkRounded;
-    private Integer positionInt = 0;
     private Color colour;
 
     public ClientYacht(String boatType, Integer sourceId, String hullID, String shortName,
@@ -146,12 +153,16 @@ public class ClientYacht extends Observable {
         this.estimateTimeAtFinish = estimateTimeAtFinish;
     }
 
-    public Integer getPositionInteger() {
+    public Integer getPlacing() {
-        return positionInt;
+        return placingProperty.get();
     }
 
-    public void setPositionInteger(Integer position) {
+    public void setPlacing(Integer position) {
-        this.positionInt = position;
+        placingProperty.set(position);
+    }
+
+    public ReadOnlyIntegerProperty placingProperty() {
+        return placingProperty.getReadOnlyProperty();
     }
 
     public void updateVelocityProperty(double velocity) {
@@ -190,6 +201,14 @@ public class ClientYacht extends Observable {
         return location;
     }
 
+    public Integer getPosition() {
+        return position;
+    }
+
+    public void setPosition(Integer position) {
+        this.position = position;
+    }
+
     public void toggleSail() {
         sailIn = !sailIn;
     }
@@ -240,14 +259,6 @@ public class ClientYacht extends Observable {
         this.colour = colour;
     }
 
-//    public Double getCurrentVelocity() {
-//        return currentVelocity;
-//    }
-//
-//    public void setCurrentVelocity(Double currentVelocity) {
-//        this.currentVelocity = currentVelocity;
-//    }
-
 
     public void updateLocation(double lat, double lng, double heading, double velocity) {
         setLocation(lat, lng);
@@ -263,7 +274,25 @@ public class ClientYacht extends Observable {
         locationListeners.add(listener);
     }
 
+    public void addMarkRoundingListener(MarkRoundingListener listener) {
+        markRoundingListeners.add(listener);
+    }
+
+    public void removeMarkRoundingListener(MarkRoundingListener listener) {
+        markRoundingListeners.remove(listener);
+    }
+
     public boolean getSailIn () {
         return sailIn;
     }
+
+    public void roundMark(CompoundMark mark, long markRoundTime, long timeSinceLastMark) {
+        this.markRoundTime = markRoundTime;
+        timeSinceLastMarkProperty.set(timeSinceLastMark);
+        lastMarkRounded = mark;
+        legNumber += 1;
+        for (MarkRoundingListener listener : markRoundingListeners) {
+            listener.notifyRounding(this, lastMarkRounded, legNumber);
+        }
+    }
 }

src/main/java/seng302/model/RaceState.java

@@ -2,7 +2,17 @@ package seng302.model;
 
 import java.text.DateFormat;
 import java.text.SimpleDateFormat;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Collection;
+import java.util.Comparator;
+import java.util.List;
+import java.util.Observable;
 import java.util.TimeZone;
+import javafx.beans.property.ReadOnlyDoubleProperty;
+import javafx.beans.property.ReadOnlyDoubleWrapper;
+import javafx.collections.FXCollections;
+import javafx.collections.ObservableList;
 import seng302.model.stream.parser.RaceStartData;
 import seng302.model.stream.parser.RaceStatusData;
 
@@ -12,22 +22,30 @@ import seng302.model.stream.parser.RaceStatusData;
  */
 public class RaceState {
 
+    @FunctionalInterface
+    public interface CollisionListener {
+        void notifyCollision(GeoPoint location);
+    }
+
 //    private final DateFormat DATE_TIME_FORMAT = new SimpleDateFormat("dd/MM/yyyy HH:mm:ss");
     private final DateFormat DATE_TIME_FORMAT = new SimpleDateFormat("HH:mm:ss");
-
+    private ReadOnlyDoubleWrapper windSpeed = new ReadOnlyDoubleWrapper();
-    private double windSpeed;
+    private ReadOnlyDoubleWrapper windDirection = new ReadOnlyDoubleWrapper();
-    private double windDirection;
     private long serverSystemTime;
     private long expectedStartTime;
     private boolean isRaceStarted = false;
     long timeTillStart;
+    private ObservableList<ClientYacht> playerPositions;
+    private List<ClientYacht> collisions = new ArrayList<>();
+    private List<CollisionListener> collisionListeners = new ArrayList<>();
 
     public RaceState() {
+        playerPositions = FXCollections.observableArrayList();
     }
 
     public void updateState (RaceStatusData data) {
-        this.windSpeed = data.getWindSpeed();
+        this.windSpeed.set(data.getWindSpeed());
-        this.windDirection = data.getWindDirection();
+        this.windDirection.set(data.getWindDirection());
         this.serverSystemTime = data.getCurrentTime();
         this.expectedStartTime = data.getExpectedStartTime();
         this.isRaceStarted = data.isRaceStarted();
@@ -55,11 +73,15 @@ public class RaceState {
     }
 
     public double getWindSpeed() {
-        return windSpeed;
+        return windSpeed.doubleValue();
     }
 
-    public double getWindDirection() {
+    public ReadOnlyDoubleProperty windSpeedProperty() {
-        return windDirection;
+        return windSpeed.getReadOnlyProperty();
+    }
+
+    public ReadOnlyDoubleProperty windDirectionProperty() {
+        return windDirection.getReadOnlyProperty();
     }
 
     public long getRaceTime() {
@@ -69,4 +91,32 @@ public class RaceState {
     public boolean isRaceStarted () {
         return isRaceStarted;
     }
+
+    public void setBoats(Collection<ClientYacht> clientYachts) {
+        playerPositions.setAll(clientYachts);
+    }
+
+    public void sortPlayers() {
+        playerPositions.sort((yacht1, yacht2) -> Integer.compare(yacht2.getLegNumber(),
+            yacht1.getLegNumber()));
+    }
+
+    public ObservableList<ClientYacht> getPlayerPositions() {
+        return playerPositions;
+    }
+
+    public void storeCollision(ClientYacht yacht) {
+        collisions.add(yacht);
+        for (CollisionListener collisionListener : collisionListeners) {
+            collisionListener.notifyCollision(yacht.getLocation());
+        }
+    }
+
+    public void addCollisionListener(CollisionListener collisionListener) {
+        collisionListeners.add(collisionListener);
+    }
+
+    public void removeCollisionListener(CollisionListener collisionListener) {
+        collisionListeners.remove(collisionListener);
+    }
 }

src/main/java/seng302/model/ServerYacht.java

@@ -43,6 +43,7 @@ public class ServerYacht extends Observable {
     private Double currentVelocity;
     private Boolean isAuto;
     private Double autoHeading;
+    private Integer legNumber;
 
     //Mark Rounding
     private Integer currentMarkSeqID;
@@ -68,6 +69,7 @@ public class ServerYacht extends Observable {
         this.heading = 120.0;   //In degrees
         this.currentVelocity = 0d;     //in mms-1
         this.currentMarkSeqID = 0;
+        this.legNumber = 0;
         this.boatColor = Colors.getColor(sourceId - 1);
 
         this.hasEnteredRoundingZone = false;
@@ -389,6 +391,14 @@ public class ServerYacht extends Observable {
         return hasPassedLine;
     }
 
+    public void incrementLegNumber() {
+        legNumber++;
+    }
+
+    public Integer getLegNumber() {
+        return legNumber;
+    }
+
     public void setBoatColor(Color color) {
         this.boatColor = color;
     }

src/main/java/seng302/model/mark/CompoundMark.java

@@ -55,7 +55,7 @@ public class CompoundMark {
 		this.name = name;
 	}
 
-    public void setRoundingSide(RoundingSide roundingSide) {
+    public void setRoundingSide(RoundingSide roundingSide) {;
         switch (roundingSide) {
             case SP:
                 getSubMark(1).setRoundingSide(RoundingSide.STARBOARD);

src/main/java/seng302/model/mark/MarkOrder.java

@@ -104,10 +104,11 @@ public class MarkOrder {
             List<Corner> corners = data.getMarkSequence();
             Map<Integer, CompoundMark> marks = data.getCompoundMarks();
             List<CompoundMark> course = new ArrayList<>();
-
             for (Corner corner : corners){
                 CompoundMark compoundMark = marks.get(corner.getCompoundMarkID());
-                compoundMark.setRoundingSide(RoundingSide.getRoundingSide(corner.getRounding()));
+                compoundMark.setRoundingSide(
+                    RoundingSide.getRoundingSide(corner.getRounding())
+                );
                 course.add(compoundMark);
                 allMarks.addAll(compoundMark.getMarks());
             }

src/main/java/seng302/utilities/StreamParser.java

@@ -95,13 +95,8 @@ public class StreamParser {
             boatID = bytesToLong(
                 Arrays.copyOfRange(payload, 24 + (i * 20), 28 + (i * 20)));
             boatStatus = (int) payload[28 + (i * 20)];
-
-//            setBoatLegPosition(boat, (int) payload[29 + (i * 20)]);
-//            boat.setPenaltiesAwarded((int) payload[30 + (i * 20)]);
-//            boat.setPenaltiesServed((int) payload[31 + (i * 20)]);
             estTimeAtNextMark = bytesToLong(
                 Arrays.copyOfRange(payload, 32 + (i * 20), 38 + (i * 20)));
-//            boat.setEstimateTimeTillNextMark(estTimeAtNextMark);
             estTimeAtFinish = bytesToLong(
                 Arrays.copyOfRange(payload, 38 + (i * 20), 44 + (i * 20)));
             leg = (int) payload[29 + (i * 20)];
@@ -119,12 +114,12 @@ public class StreamParser {
 //                    placing += 1;
 //                }
 //            }
-//            updatingBoat.setPosition(placing.toString());
+//            updatingBoat.setPlacing(placing.toString());
 //            updatingBoat.setLegNumber(leg);
 //            boatsPos.putIfAbsent(placing, updatingBoat);
 //            boatsPos.replace(placing, updatingBoat);
 //        } else if(updatingBoat.getLegNumber() == null){
-//            updatingBoat.setPosition("1");
+//            updatingBoat.setPlacing("1");
 //            updatingBoat.setLegNumber(leg);
 //        }
 //    }

src/main/java/seng302/visualiser/GameClient.java

@@ -15,6 +15,8 @@ import javafx.scene.layout.Pane;
 import seng302.gameServer.GameState;
 import seng302.gameServer.MainServerThread;
 import seng302.gameServer.messages.BoatAction;
+import seng302.gameServer.messages.BoatStatus;
+import seng302.gameServer.messages.BoatAction;
 import seng302.model.ClientYacht;
 import seng302.model.RaceState;
 import seng302.model.stream.packets.StreamPacket;
@@ -27,6 +29,7 @@ import seng302.model.stream.xml.parser.RaceXMLData;
 import seng302.model.stream.xml.parser.RegattaXMLData;
 import seng302.utilities.StreamParser;
 import seng302.utilities.XMLParser;
+import seng302.visualiser.controllers.FinishScreenViewController;
 import seng302.visualiser.controllers.LobbyController;
 import seng302.visualiser.controllers.LobbyController.CloseStatus;
 import seng302.visualiser.controllers.RaceViewController;
@@ -169,17 +172,7 @@ public class GameClient {
     }
 
     private void loadRaceView() {
-        FXMLLoader fxmlLoader = new FXMLLoader(
+        FXMLLoader fxmlLoader = loadFXMLToHolder("/views/RaceView.fxml");
-            RaceViewController.class.getResource("/views/RaceView.fxml"));
-        try {
-            final Node node = fxmlLoader.load();
-            Platform.runLater(() -> {
-                holderPane.getChildren().clear();
-                holderPane.getChildren().add(node);
-            });
-        } catch (IOException e) {
-            e.printStackTrace();
-        }
         holderPane.getScene().setOnKeyPressed(this::keyPressed);
         holderPane.getScene().setOnKeyReleased(this::keyReleased);
         raceView = fxmlLoader.getController();
@@ -188,17 +181,25 @@ public class GameClient {
     }
 
     private void loadFinishScreenView() {
+        FXMLLoader fxmlLoader = loadFXMLToHolder("/views/FinishScreenView.fxml");
+        FinishScreenViewController controller = fxmlLoader.getController();
+        controller.setFinishers(raceState.getPlayerPositions());
+    }
+
+    private FXMLLoader loadFXMLToHolder(String fxmlLocation) {
         FXMLLoader fxmlLoader = new FXMLLoader(
-            getClass().getResource("/views/FinishScreenView.fxml"));
+            getClass().getResource(fxmlLocation)
+        );
         try {
-            final Node finishScreenFX = fxmlLoader.load();
+            final Node fxmlLoaderFX = fxmlLoader.load();
             Platform.runLater(() -> {
                 holderPane.getChildren().clear();
-                holderPane.getChildren().add(finishScreenFX);
+                holderPane.getChildren().add(fxmlLoaderFX);
             });
         } catch (IOException e) {
             e.printStackTrace();
         }
+        return fxmlLoader;
     }
 
     private void parsePackets() {
@@ -243,6 +244,7 @@ public class GameClient {
                     allBoatsMap.forEach((id, boat) ->
                         clientLobbyList.add(boat.getBoatName())
                     );
+                    raceState.setBoats(allBoatsMap.values());
                     break;
 
                 case RACE_START_STATUS:
@@ -281,8 +283,8 @@ public class GameClient {
     private void updatePosition(PositionUpdateData positionData) {
         if (positionData.getType() == DeviceType.YACHT_TYPE) {
             if (allXMLReceived() && allBoatsMap.containsKey(positionData.getDeviceId())) {
-                ClientYacht clientYacht = allBoatsMap.get(positionData.getDeviceId());
+                ClientYacht yacht = allBoatsMap.get(positionData.getDeviceId());
-                clientYacht.updateLocation(positionData.getLat(),
+                yacht.updateLocation(positionData.getLat(),
                     positionData.getLon(), positionData.getHeading(),
                     positionData.getGroundSpeed());
             }
@@ -300,13 +302,10 @@ public class GameClient {
     private void updateMarkRounding(MarkRoundingData roundingData) {
         if (allXMLReceived()) {
             ClientYacht clientYacht = allBoatsMap.get(roundingData.getBoatId());
-            clientYacht.setMarkRoundingTime(roundingData.getTimeStamp());
+            clientYacht.roundMark(
-            clientYacht.updateTimeSinceLastMarkProperty(
+                courseData.getCompoundMarks().get(roundingData.getMarkId()),
-                raceState.getRaceTime() - roundingData.getTimeStamp());
+                roundingData.getTimeStamp(),
-            clientYacht.setLastMarkRounded(
+                raceState.getRaceTime() - roundingData.getTimeStamp()
-                courseData.getCompoundMarks().get(
-                    roundingData.getMarkId()
-                )
             );
         }
     }
@@ -314,37 +313,36 @@ public class GameClient {
     private void processRaceStatusUpdate(RaceStatusData data) {
         if (allXMLReceived()) {
             raceState.updateState(data);
-            if (raceView != null) {
-                raceView.getGameView().setWindDir(raceState.getWindDirection());
-            }
             boolean raceFinished = true;
             for (ClientYacht yacht : allBoatsMap.values()) {
-                if (yacht.getBoatStatus() != 3) {
+                if (yacht.getBoatStatus() != BoatStatus.FINISHED.getCode()) {
                     raceFinished = false;
                 }
             }
-            if (raceFinished == true) {
-                close();
-                loadFinishScreenView();
-            }
 
             for (long[] boatData : data.getBoatData()) {
                 ClientYacht clientYacht = allBoatsMap.get((int) boatData[0]);
                 clientYacht.setEstimateTimeTillNextMark(raceState.getRaceTime() - boatData[1]);
                 clientYacht.setEstimateTimeAtFinish(boatData[2]);
                 int legNumber = (int) boatData[3];
-                clientYacht.setLegNumber(legNumber);
                 clientYacht.setBoatStatus((int) boatData[4]);
                 if (legNumber != clientYacht.getLegNumber()) {
-                    int placing = 1;
+                    clientYacht.setLegNumber(legNumber);
-                    for (ClientYacht otherClientYacht : allBoatsMap.values()) {
+                    updatePlayerPositions();
-                        if (otherClientYacht.getSourceId() != boatData[0] &&
-                            clientYacht.getLegNumber() <= otherClientYacht.getLegNumber())
-                            placing++;
-                    }
-                    clientYacht.setPositionInteger(placing);
                 }
             }
+
+            if (raceFinished) {
+                close();
+                loadFinishScreenView();
+            }
+        }
+    }
+
+    private void updatePlayerPositions() {
+        raceState.sortPlayers();
+        for (ClientYacht yacht : raceState.getPlayerPositions()) {
+            yacht.setPosition(raceState.getPlayerPositions().indexOf(yacht) + 1);
         }
     }
 
@@ -367,22 +365,16 @@ public class GameClient {
                 socketThread.sendBoatAction(BoatAction.DOWNWIND); break;
             case ENTER: // tack/gybe
                 socketThread.sendBoatAction(BoatAction.TACK_GYBE); break;
-            //TODO Allow a zoom in and zoom out methods
-            case Z:  // zoom in
-                System.out.println("Zoom in");
-                break;
-            case X:  // zoom out
-                System.out.println("Zoom out");
-                break;
         }
     }
 
+
     private void keyReleased(KeyEvent e) {
         switch (e.getCode()) {
             //TODO 12/07/17 Determine the sail state and send the appropriate packet (eg. if sails are in, send a sail out packet)
             case SHIFT:  // sails in/sails out
                 socketThread.sendBoatAction(BoatAction.SAILS_IN);
-                raceView.getGameView().getPlayerYacht().toggleSail();
+                allBoatsMap.get(socketThread.getClientId()).toggleSail();
                 break;
             case PAGE_UP:
             case PAGE_DOWN:
@@ -400,7 +392,11 @@ public class GameClient {
     private void showCollisionAlert(YachtEventData yachtEventData) {
         // 33 is the agreed code to show collision
         if (yachtEventData.getEventId() == 33) {
-            raceView.showCollision(yachtEventData.getSubjectId());
+            raceState.storeCollision(
+                allBoatsMap.get(
+                    yachtEventData.getSubjectId().intValue()
+                )
+            );
         }
     }
 }

src/main/java/seng302/visualiser/GameView.java

@@ -16,6 +16,8 @@ import javafx.geometry.Point2D;
 import javafx.scene.Group;
 import javafx.scene.Node;
 import javafx.scene.image.ImageView;
+import javafx.scene.input.KeyCode;
+import javafx.scene.input.KeyEvent;
 import javafx.scene.layout.AnchorPane;
 import javafx.scene.layout.Pane;
 import javafx.scene.paint.Color;
@@ -25,6 +27,9 @@ import javafx.scene.shape.Polygon;
 import javafx.scene.text.Text;
 import javafx.util.Duration;
 import seng302.model.ClientYacht;
+import seng302.gameServer.messages.RoundingSide;
+import seng302.model.ClientYacht;
+import seng302.model.Colors;
 import seng302.model.GeoPoint;
 import seng302.model.Limit;
 import seng302.model.mark.CompoundMark;
@@ -35,6 +40,7 @@ import seng302.visualiser.fxObjects.AnnotationBox;
 import seng302.visualiser.fxObjects.BoatObject;
 import seng302.visualiser.fxObjects.CourseBoundary;
 import seng302.visualiser.fxObjects.Gate;
+import seng302.visualiser.fxObjects.MarkArrowFactory;
 import seng302.visualiser.fxObjects.Marker;
 import seng302.visualiser.map.Boundary;
 import seng302.visualiser.map.CanvasMap;
@@ -70,11 +76,13 @@ public class GameView extends Pane {
     private Map<ClientYacht, BoatObject> boatObjects = new HashMap<>();
     private Map<ClientYacht, AnnotationBox> annotations = new HashMap<>();
     private ObservableList<Node> gameObjects;
+    private BoatObject selectedBoat = null;
     private Group annotationsGroup = new Group();
     private Group wakesGroup = new Group();
     private Group boatObjectGroup = new Group();
     private Group trails = new Group();
     private Group markers = new Group();
+    private List<CompoundMark> course = new ArrayList<>();
 
     private ImageView mapImage = new ImageView();
 
@@ -91,19 +99,19 @@ public class GameView extends Pane {
 
     double scaleFactor = 1;
 
-    public void zoomOut() {
+    private void zoomOut() {
-        scaleFactor = 0.95;
+        scaleFactor = 0.1;
-        for (Node child : getChildren()) {
+        if (this.getScaleX() > 0.5) {
-            child.setScaleX(child.getScaleX() * scaleFactor);
+            this.setScaleX(this.getScaleX() - scaleFactor);
-            child.setScaleY(child.getScaleY() * scaleFactor);
+            this.setScaleY(this.getScaleY() - scaleFactor);
         }
     }
 
-    public void zoomIn() {
+    private void zoomIn() {
-        scaleFactor =  1.05;
+        scaleFactor = 0.10;
-        for (Node child : getChildren()) {
+        if (this.getScaleX() < 2.5) {
-            child.setScaleX(child.getScaleX() * scaleFactor);
+            this.setScaleX(this.getScaleX() + scaleFactor);
-            child.setScaleY(child.getScaleY() * scaleFactor);
+            this.setScaleY(this.getScaleY() + scaleFactor);
         }
     }
 
@@ -112,14 +120,25 @@ public class GameView extends Pane {
         VERTICAL
     }
 
-    public GameView() {
+
+    private void trackBoat() {
+        if (selectedBoat != null) {
+            double x = selectedBoat.getBoatLayoutX();
+            double y = selectedBoat.getBoatLayoutY();
+            double displacementX = this.getWidth();
+            double displacementY = this.getHeight();
+            this.setLayoutX((-x + (displacementX / 2.0)) * this.getScaleX());
+            this.setLayoutY((-y + (displacementY / 2.0)) * this.getScaleY());
+        } else {
+            this.setLayoutX(0);
+            this.setLayoutY(0);
+        }
+    }
+
+    public GameView () {
         gameObjects = this.getChildren();
         // create image view for map, bind panel size to image
         gameObjects.add(mapImage);
-        fpsDisplay.setLayoutX(5);
-        fpsDisplay.setLayoutY(20);
-        fpsDisplay.setStrokeWidth(2);
-        gameObjects.add(fpsDisplay);
         gameObjects.add(raceBorder);
         gameObjects.add(markers);
         initializeTimer();
@@ -137,6 +156,7 @@ public class GameView extends Pane {
 
             @Override
             public void handle(long now) {
+                trackBoat();
                 if (lastTime == 0) {
                     lastTime = now;
                 } else {
@@ -160,9 +180,7 @@ public class GameView extends Pane {
                         lastTime = now;
                     }
                 }
-//                Platform.runLater(() ->
+                boatObjects.forEach((boat, boatObject) -> boatObject.updateLocation());
-                    boatObjects.forEach((boat, boatObject) -> boatObject.updateLocation());
-//                );
             }
         };
     }
@@ -204,6 +222,7 @@ public class GameView extends Pane {
         mapImage.fitHeightProperty().bind(((AnchorPane) this.getParent()).heightProperty());
     }
 
+    // TODO: 16/08/17 Break up this function
     /**
      * Adds a course to the GameView. The view is scaled accordingly unless a border is set in which
      * case the course is added relative ot the border.
@@ -213,6 +232,23 @@ public class GameView extends Pane {
      */
     public void updateCourse(List<CompoundMark> newCourse, List<Corner> sequence) {
         markerObjects = new HashMap<>();
+
+        for (Corner corner : sequence) { //Makes course out of all compound marks.
+            for (CompoundMark compoundMark : newCourse) {
+                if (corner.getCompoundMarkID() == compoundMark.getId()) {
+                    course.add(compoundMark);
+                }
+            }
+        }
+
+        // TODO: 16/08/17 Updating mark roundings here. It should not happen here. Nor should it be done this way.
+        for (Corner corner : sequence){
+            CompoundMark compoundMark = course.get(corner.getSeqID() - 1);
+            compoundMark.setRoundingSide(
+                RoundingSide.getRoundingSide(corner.getRounding())
+            );
+        }
+
         final List<Gate> gates = new ArrayList<>();
         Paint colour = Color.BLACK;
         //Creates new markers
@@ -227,16 +263,6 @@ public class GameView extends Pane {
             for (Mark mark : cMark.getMarks()) {
                 makeAndBindMarker(mark, colour);
             }
-
-            //UNCOMMENT THIS TO HIGHLIGHT SUBMARKS 1 and 2 RED AND GREEN RESPECTIVELY FOR DEBUG
-            //(instead of above for loop)
-//            for (Mark mark : cMark.getMarks()) {
-//                if (mark.getSeqID() == 1) {
-//                    makeAndBindMarker(mark, Color.RED);
-//                } else {
-//                    makeAndBindMarker(mark, Color.GREEN);
-//                }
-//            }
             //Create gate line
             if (cMark.isGate()) {
                 for (int i = 1; i < cMark.getMarks().size(); i++) {
@@ -251,6 +277,73 @@ public class GameView extends Pane {
             }
             colour = Color.BLACK;
         }
+
+        //Creating mark arrows.
+        for (int i=1; i < sequence.size()-1; i++) { //General case.
+            double averageLat = 0;
+            double averageLng = 0;
+            int numMarks = 0;
+            for (Mark mark : course.get(i-1).getMarks()) {
+                numMarks += 1;
+                averageLat += mark.getLat();
+                averageLng += mark.getLng();
+            }
+            GeoPoint lastMarkAv = new GeoPoint(averageLat / numMarks, averageLng / numMarks);
+            numMarks = 0;
+            averageLat = 0;
+            averageLng = 0;
+            for (Mark mark : course.get(i+1).getMarks()) {
+                numMarks += 1;
+                averageLat += mark.getLat();
+                averageLng += mark.getLng();
+            }
+            GeoPoint nextMarkAv = new GeoPoint(averageLat / numMarks, averageLng / numMarks);
+            // TODO: 16/08/17 This comparison is cancer and deserves to die.
+            for (Mark mark : course.get(i).getMarks()) {
+                markerObjects.get(mark).addArrows(
+                    mark.getRoundingSide() == RoundingSide.STARBOARD ? MarkArrowFactory.RoundingSide.STARBOARD : MarkArrowFactory.RoundingSide.PORT,
+                    GeoUtility.getBearing(lastMarkAv, mark),
+                    GeoUtility.getBearing(mark, nextMarkAv)
+                );
+            }
+        }
+
+        // TODO: 16/08/17 Make this cleaner
+        //First mark case
+        double averageLat = 0;
+        double averageLng = 0;
+        int numMarks = 0;
+        for (Mark mark : course.get(1).getMarks()) {
+            numMarks += 1;
+            averageLat += mark.getLat();
+            averageLng += mark.getLng();
+        }
+        GeoPoint firstMarkAv = new GeoPoint(averageLat / numMarks, averageLng / numMarks);
+        for (Mark mark : course.get(0).getMarks()) {
+            markerObjects.get(mark).addArrows(
+                mark.getRoundingSide() == RoundingSide.STARBOARD ? MarkArrowFactory.RoundingSide.STARBOARD : MarkArrowFactory.RoundingSide.PORT,
+                0d, //90
+                GeoUtility.getBearing(mark, firstMarkAv)
+            );
+        }
+        //Last Mark case
+        numMarks = 0;
+        averageLat = 0;
+        averageLng = 0;
+        for (Mark mark : course.get(course.size()-2).getMarks()) {
+            numMarks += 1;
+            averageLat += mark.getLat();
+            averageLng += mark.getLng();
+        }
+        GeoPoint secondToLastMarkAv = new GeoPoint(averageLat / numMarks, averageLng / numMarks);
+        for (Mark mark : course.get(course.size()-1).getMarks()) {
+            markerObjects.get(mark).addArrows(
+                mark.getRoundingSide() == RoundingSide.STARBOARD ? MarkArrowFactory.RoundingSide.STARBOARD : MarkArrowFactory.RoundingSide.PORT,
+                GeoUtility.getBearing(secondToLastMarkAv, mark),
+                GeoUtility.getBearing(mark, mark)
+            );
+        }
+
         //Scale race to markers if there is no border.
         if (borderPoints == null) {
             rescaleRace(new ArrayList<>(markerObjects.keySet()));
@@ -258,8 +351,8 @@ public class GameView extends Pane {
         //Move the Markers to initial position.
         markerObjects.forEach(((mark, marker) -> {
             Point2D p2d = findScaledXY(mark.getLat(), mark.getLng());
-            marker.setCenterX(p2d.getX());
+            marker.setLayoutX(p2d.getX());
-            marker.setCenterY(p2d.getY());
+            marker.setLayoutY(p2d.getY());
         }));
         Platform.runLater(() -> {
             markers.getChildren().clear();
@@ -276,11 +369,12 @@ public class GameView extends Pane {
      */
     private void makeAndBindMarker(Mark observableMark, Paint colour) {
         Marker marker = new Marker(colour);
+//        marker.addArrows(MarkArrowFactory.RoundingSide.PORT, ThreadLocalRandom.current().nextDouble(91, 180), ThreadLocalRandom.current().nextDouble(1, 90));
         markerObjects.put(observableMark, marker);
         observableMark.addPositionListener((mark, lat, lon) -> {
             Point2D p2d = findScaledXY(lat, lon);
-            markerObjects.get(mark).setCenterX(p2d.getX());
+            markerObjects.get(mark).setLayoutX(p2d.getX());
-            markerObjects.get(mark).setCenterY(p2d.getY());
+            markerObjects.get(mark).setLayoutY(p2d.getY());
         });
     }
 
@@ -295,16 +389,16 @@ public class GameView extends Pane {
     private Gate makeAndBindGate(Marker m1, Marker m2, Paint colour) {
         Gate gate = new Gate(colour);
         gate.startXProperty().bind(
-            m1.centerXProperty()
+            m1.layoutXProperty()
         );
         gate.startYProperty().bind(
-            m1.centerYProperty()
+            m1.layoutYProperty()
         );
         gate.endXProperty().bind(
-            m2.centerXProperty()
+            m2.layoutXProperty()
         );
         gate.endYProperty().bind(
-            m2.centerYProperty()
+            m2.layoutYProperty()
         );
         return gate;
     }
@@ -329,6 +423,21 @@ public class GameView extends Pane {
         raceBorder.getPoints().setAll(boundaryPoints);
     }
 
+    // TODO: 16/08/17 initialize zooming internal to GameView only
+    /**
+     * Enables zoom. Has to be called after this is added to a scene.
+     */
+    public void enableZoom () {
+        if (this.getScene() != null) {
+            this.getScene().addEventHandler(KeyEvent.KEY_PRESSED, (event) -> {
+                if (event.getCode() == KeyCode.Z) {
+                    zoomIn();
+                } else if (event.getCode() == KeyCode.X) {
+                    zoomOut();
+                }
+            });
+        }
+    }
     /**
      * Rescales the race to the size of the window.
      *
@@ -342,16 +451,33 @@ public class GameView extends Pane {
 //        drawGoogleMap();
     }
 
+    private void setSelectedBoat(BoatObject bo, Boolean isSelected) {
+        if (this.selectedBoat == bo && !isSelected) {
+            this.selectedBoat = null;
+            boatObjects.forEach((boat, group) ->
+                group.setIsSelected(false)
+            );
+        } else if (isSelected) {
+            this.selectedBoat = bo;
+            for (BoatObject group : boatObjects.values()) {
+                if (group != bo) {
+                    group.setIsSelected(false);
+                }
+            }
+        }
+    }
+
     /**
      * Draws all the boats.
-     * @param  clientYachts The yachts to set in the race
+     * @param  yachts The yachts to set in the race
      */
-    public void setBoats(List<ClientYacht> clientYachts) {
+    public void setBoats(List<ClientYacht> yachts) {
         BoatObject newBoat;
         final List<Group> wakes = new ArrayList<>();
         for (ClientYacht clientYacht : clientYachts) {
             Paint colour = clientYacht.getColour();
             newBoat = new BoatObject();
+            newBoat.addSelectedBoatListener(this::setSelectedBoat);
             newBoat.setFill(colour);
             boatObjects.put(clientYacht, newBoat);
             createAndBindAnnotationBox(clientYacht, colour);
@@ -364,9 +490,6 @@ public class GameView extends Pane {
                 BoatObject bo = boatObjects.get(boat);
                 Point2D p2d = findScaledXY(lat, lon);
                 bo.moveTo(p2d.getX(), p2d.getY(), heading, velocity, sailIn, windDir);
-//                annotations.get(boat).setLayoutX(p2d.getX());
-//                annotations.get(boat).setLayoutY(p2d.getY());
-//                annotations.get(boat).setLocation(100d, 100d);
                 annotations.get(boat).setLocation(p2d.getX(), p2d.getY());
                 bo.setTrajectory(
                     heading,
@@ -613,7 +736,6 @@ public class GameView extends Pane {
         timer.stop();
     }
 
-
     public void setWindDir(double windDir) {
         this.windDir = windDir;
     }
@@ -626,10 +748,14 @@ public class GameView extends Pane {
         return playerYacht;
     }
 
-
     public void setBoatAsPlayer (ClientYacht playerYacht) {
         this.playerYacht = playerYacht;
+        playerYacht.toggleSail();
         boatObjects.get(playerYacht).setAsPlayer();
+        CompoundMark currentMark = course.get(playerYacht.getLegNumber());
+        for (Mark mark : currentMark.getMarks()) {
+            markerObjects.get(mark).showNextExitArrow();
+        }
         annotations.get(playerYacht).addAnnotation(
             "velocity",
             playerYacht.getVelocityProperty(),
@@ -641,6 +767,31 @@ public class GameView extends Pane {
             annotationsGroup.getChildren().remove(annotations.get(playerYacht));
             gameObjects.add(annotations.get(playerYacht));
         });
+        playerYacht.addMarkRoundingListener(this::updateMarkArrows);
+    }
+
+    private void updateMarkArrows (ClientYacht yacht, CompoundMark compoundMark, int legNumber) {
+        //Only show arrows for this and next leg.
+        if (compoundMark != null) {
+            for (Mark mark : compoundMark.getMarks()) {
+                markerObjects.get(mark).showNextExitArrow();
+            }
+        }
+        CompoundMark nextMark = null;
+        if (legNumber < course.size() - 1) {
+            nextMark = course.get(legNumber);
+            for (Mark mark : nextMark.getMarks()) {
+                markerObjects.get(mark).showNextEnterArrow();
+            }
+        }
+        if (legNumber - 2 >= 0) {
+            CompoundMark lastMark = course.get(Math.max(0, legNumber - 2));
+            if (lastMark != nextMark) {
+                for (Mark mark : lastMark.getMarks()) {
+                    markerObjects.get(mark).hideAllArrows();
+                }
+            }
+        }
     }
 
     /**
@@ -650,32 +801,35 @@ public class GameView extends Pane {
      * @param collisionPoint yacht collision point
      */
     public void drawCollision(GeoPoint collisionPoint) {
-        Platform.runLater(() -> {
+        Point2D point = findScaledXY(collisionPoint);
-            Point2D point = findScaledXY(collisionPoint);
+        double circleRadius = 0.0;
-            double circleRadius = 0.0;
+        Circle circle = new Circle(point.getX(), point.getY(), circleRadius, Color.RED);
-            Circle circle = new Circle(point.getX(), point.getY(), circleRadius, Color.RED);
-            gameObjects.add(circle);
 
-            circle.setFill(Color.TRANSPARENT);
+        circle.setFill(Color.TRANSPARENT);
-            circle.setStroke(Color.RED);
+        circle.setStroke(Color.RED);
-            circle.setStrokeWidth(3);
+        circle.setStrokeWidth(3);
 
-            Timeline timeline = new Timeline();
+        Timeline timeline = new Timeline();
-            timeline.setCycleCount(1);
+        timeline.setCycleCount(1);
 
-            KeyFrame keyframe1 = new KeyFrame(Duration.ZERO,
+        KeyFrame keyframe1 = new KeyFrame(Duration.ZERO,
-                new KeyValue(circle.radiusProperty(), 0),
+            new KeyValue(circle.radiusProperty(), 0),
-                new KeyValue(circle.strokeProperty(), Color.TRANSPARENT));
+            new KeyValue(circle.strokeProperty(), Color.TRANSPARENT));
-            KeyFrame keyFrame2 = new KeyFrame(new Duration(1000),
+        KeyFrame keyFrame2 = new KeyFrame(new Duration(1000),
-                new KeyValue(circle.radiusProperty(), 50),
+            new KeyValue(circle.radiusProperty(), 50),
-                new KeyValue(circle.strokeProperty(), Color.RED));
+            new KeyValue(circle.strokeProperty(), Color.RED));
-            KeyFrame keyFrame3 = new KeyFrame(new Duration(1500),
+        KeyFrame keyFrame3 = new KeyFrame(new Duration(1500),
-                new KeyValue(circle.strokeProperty(), Color.TRANSPARENT));
+            new KeyValue(circle.strokeProperty(), Color.TRANSPARENT));
 
-            timeline.getKeyFrames().addAll(keyframe1, keyFrame2, keyFrame3);
+        timeline.getKeyFrames().addAll(keyframe1, keyFrame2, keyFrame3);
-            timeline.play();
 
-            timeline.setOnFinished(event -> gameObjects.remove(circle));
+        Platform.runLater(() -> gameObjects.add(circle));
-        });
+        timeline.setOnFinished(event -> Platform.runLater(() -> gameObjects.remove(circle)));
+        timeline.play();
+    }
+
+    public void setFrameRateFXText(Text fpsDisplay) {
+        this.fpsDisplay = null;
+        this.fpsDisplay = fpsDisplay;
     }
 }

src/main/java/seng302/visualiser/controllers/FinishScreenViewController.java

@@ -3,6 +3,7 @@ package seng302.visualiser.controllers;
 import java.io.IOException;
 import java.net.URL;
 import java.util.ArrayList;
+import java.util.Collection;
 import java.util.Comparator;
 import java.util.List;
 import java.util.ResourceBundle;
@@ -61,9 +62,9 @@ public class FinishScreenViewController implements Initializable {
         finishOrderTable.refresh();
     }
 
-    public void setFinishers(List<ClientYacht> participants) {
+    public void setFinishers(Collection<ClientYacht> participants) {
         List<ClientYacht> sorted = new ArrayList<>(participants);
-        sorted.sort(Comparator.comparingInt(ClientYacht::getPositionInteger));
+        sorted.sort(Comparator.comparingInt(ClientYacht::getPlacing));
         finishOrderTable.getItems().setAll(sorted);
     }
 

src/main/java/seng302/visualiser/controllers/RaceViewController.java

@@ -2,7 +2,6 @@ package seng302.visualiser.controllers;
 
 import java.io.IOException;
 import java.util.ArrayList;
-import java.util.Comparator;
 import java.util.List;
 import java.util.Map;
 import java.util.Timer;
@@ -11,6 +10,8 @@ import java.util.concurrent.TimeUnit;
 import javafx.animation.Timeline;
 import javafx.application.Platform;
 import javafx.collections.FXCollections;
+import javafx.collections.ListChangeListener;
+import javafx.collections.ObservableList;
 import javafx.fxml.FXML;
 import javafx.fxml.FXMLLoader;
 import javafx.geometry.Point2D;
@@ -71,6 +72,10 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
     private Button selectAnnotationBtn;
     @FXML
     private ComboBox<ClientYacht> yachtSelectionComboBox;
+    @FXML
+    private Text fpsDisplay;
+    @FXML
+    private Text windSpeedText;
 
     //Race Data
     private Map<Integer, ClientYacht> participants;
@@ -79,6 +84,7 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
     private GameView gameView;
     private RaceState raceState;
 
+    private Timeline timerTimeline;
     private Timer timer = new Timer();
     private List<Series<String, Double>> sparkLineData = new ArrayList<>();
     private ImportantAnnotationsState importantAnnotations;
@@ -101,8 +107,8 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 
     public void loadRace (
         Map<Integer, ClientYacht> participants, RaceXMLData raceData, RaceState raceState,
-        ClientYacht player
+        ClientYacht player) {
-    ) {
+
         this.participants = participants;
         this.courseData = raceData;
         this.markers = raceData.getCompoundMarks();
@@ -114,15 +120,38 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
         initialiseBoatSelectionComboBox();
         initialiseSparkLine();
 
+        raceState.getPlayerPositions().addListener((ListChangeListener<ClientYacht>) c -> {
+            while (c.next()) {
+                if (c.wasPermutated()) {
+                    updateOrder(raceState.getPlayerPositions());
+                }
+            }
+        });
+
+        updateOrder(raceState.getPlayerPositions());
         gameView = new GameView();
-        Platform.runLater(() -> contentAnchorPane.getChildren().add(gameView));
+        gameView.setFrameRateFXText(fpsDisplay);
-        gameView.setBoats(new ArrayList<>(participants.values()));
+        Platform.runLater(() -> contentAnchorPane.getChildren().add(0, gameView));
-        gameView.updateBorder(raceData.getCourseLimit());
+            gameView.setBoats(new ArrayList<>(participants.values()));
-        gameView.updateCourse(
+            gameView.updateBorder(raceData.getCourseLimit());
-            new ArrayList<>(raceData.getCompoundMarks().values()), raceData.getMarkSequence()
+            gameView.updateCourse(
+                new ArrayList<>(raceData.getCompoundMarks().values()), raceData.getMarkSequence()
         );
+        gameView.enableZoom();
         gameView.setBoatAsPlayer(player);
         gameView.startRace();
+
+        raceState.addCollisionListener(gameView::drawCollision);
+        raceState.windDirectionProperty().addListener((obs, oldDirection, newDirection) -> {
+            gameView.setWindDir(newDirection.doubleValue());
+            updateWindDirection(newDirection.doubleValue());
+        });
+        raceState.windSpeedProperty().addListener((obs, oldSpeed, newSpeed) -> {
+            updateWindSpeed(newSpeed.doubleValue());
+        });
+        updateWindDirection(raceState.windDirectionProperty().doubleValue());
+        updateWindSpeed(raceState.getWindSpeed());
+        gameView.setWindDir(raceState.windDirectionProperty().doubleValue());
     }
 
     /**
@@ -208,10 +237,9 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 
 
     /**
-     * Used to add any new yachts into the race that may have started late or not have had data
+     * Used to add any new yachts into the race that may have started late or not have had data received yet
-     * received yet
      */
-    private void updateSparkLine() {
+    private void updateSparkLine(){
         // TODO: 2/08/17 there is about 0 chance of this working. Once we are keeping track of boat positions it can be fixed.
         // Collect the racing yachts that aren't already in the chart
         sparkLineData.clear();
@@ -219,40 +247,39 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
         // Create a new data series for new yachts
         sparkLineCandidates
             .stream()
-            .filter(yacht -> yacht.getPositionInteger() != null)
+            .filter(yacht -> yacht.getPlacing() != null)
             .forEach(yacht -> {
                 Series<String, Double> yachtData = new Series<>();
                 yachtData.setName(yacht.getSourceId().toString());
                 yachtData.getData().add(
                     new XYChart.Data<>(
                         Integer.toString(yacht.getLegNumber()),
-                        1.0 + participants.size() - yacht.getPositionInteger()
+                        1.0 + participants.size() - yacht.getPlacing()
                     )
                 );
-                sparkLineData.add(yachtData);
+            sparkLineData.add(yachtData);
             });
 
         // Lambda function to sort the series in order of leg (later legs shown more to the right)
         sparkLineData.sort((o1, o2) -> {
-            Integer leg1 = Integer.parseInt(o1.getData().get(o1.getData().size() - 1).getXValue());
+            Integer leg1 =  Integer.parseInt(o1.getData().get(o1.getData().size()-1).getXValue());
-            Integer leg2 = Integer.parseInt(o2.getData().get(o2.getData().size() - 1).getXValue());
+            Integer leg2 =  Integer.parseInt(o2.getData().get(o2.getData().size()-1).getXValue());
-            if (leg2 < leg1) {
+            if (leg2 < leg1){
                 return 1;
             } else {
                 return -1;
             }
         });
         // Adds the new data series to the sparkline (and set the colour of the series)
-        Platform.runLater(() ->
+        Platform.runLater(() -> {
             sparkLineData
                 .stream()
                 .filter(spark -> !raceSparkLine.getData().contains(spark))
                 .forEach(spark -> {
                     raceSparkLine.getData().add(spark);
-                    spark.getNode().lookup(".chart-series-line")
+                    spark.getNode().lookup(".chart-series-line").setStyle("-fx-stroke:" + getBoatColorAsRGB(spark.getName()));
-                        .setStyle("-fx-stroke:" + getBoatColorAsRGB(spark.getName()));
+                });
-                })
+        });
-        );
     }
 
     private void initialiseSparkLine() {
@@ -262,15 +289,15 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 
     /**
      * Updates the yachts sparkline of the desired yacht and using the new leg number
-     * @param clientYacht The yacht to be updated on the sparkline
+     * @param yacht The yacht to be updated on the sparkline
      * @param legNumber the leg number that the position will be assigned to
      */
-    void updateYachtPositionSparkline(ClientYacht clientYacht, Integer legNumber) {
+    void updateYachtPositionSparkline(ClientYacht yacht, Integer legNumber){
         for (XYChart.Series<String, Double> positionData : sparkLineData) {
             positionData.getData().add(
                 new Data<>(
                     Integer.toString(legNumber),
-                    1.0 + participants.size() - clientYacht.getPositionInteger()
+                    1.0 + participants.size() - yacht.getPlacing()
                 )
             );
         }
@@ -278,7 +305,7 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 //        positionData.getData().add(
 //            new XYChart.Data<>(
 //                Integer.toString(legNumber),
-//                1.0 + participants.size() - yacht.getPosition()
+//                1.0 + participants.size() - yacht.getPlacing()
 //            )
 //        );
     }
@@ -286,52 +313,47 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 
     /**
      * gets the rgb string of the yachts colour to use for the chart via css
-     *
      * @param yachtId id of yacht passed in to get the yachts colour
      * @return the colour as an rgb string
      */
-    private String getBoatColorAsRGB(String yachtId) {
+    private String getBoatColorAsRGB(String yachtId){
         Color color = participants.get(Integer.valueOf(yachtId)).getColour();
-        if (color == null) {
+        if (color == null){
-            return String.format("#%02X%02X%02X", 255, 255, 255);
+            return String.format("#%02X%02X%02X",255,255,255);
         }
-        return String.format("#%02X%02X%02X",
+        return String.format( "#%02X%02X%02X",
-            (int) (color.getRed() * 255),
+            (int)( color.getRed() * 255 ),
-            (int) (color.getGreen() * 255),
+            (int)( color.getGreen() * 255 ),
-            (int) (color.getBlue() * 255)
+            (int)( color.getBlue() * 255 )
         );
     }
 
 
     /**
      * Initialises a timer which updates elements of the RaceView such as wind direction, yacht
-     * orderings etc.. which are dependent on the info from the stream parser constantly. Updates of
+     * orderings etc.. which are dependent on the info from the stream parser constantly.
-     * each of these attributes are called ONCE EACH SECOND
+     * Updates of each of these attributes are called ONCE EACH SECOND
      */
     private void initializeUpdateTimer() {
         timer.scheduleAtFixedRate(new TimerTask() {
             @Override
             public void run() {
                 updateRaceTime();
-                updateWindDirection();
-                updateOrder();
-//                updateSparkLine();
             }
         }, 0, 1000);
     }
 
     /**
-     * Iterates over all corners until ones SeqID matches with the yachts current leg number. Then
+     * Iterates over all corners until ones SeqID matches with the yachts current leg number.
-     * it gets the compoundMarkID of that corner and uses it to fetch the appropriate mark Returns
+     * Then it gets the compoundMarkID of that corner and uses it to fetch the appropriate mark
-     * null if no next mark found.
+     * Returns null if no next mark found.
-     *
      * @param bg The BoatGroup to find the next mark of
      * @return The next Mark or null if none found
      */
     private Mark getNextMark(BoatObject bg) {
         // TODO: 1/08/17 Move to GameView
 //
-//        Integer legNumber = bg.getYacht().getLegNumber();
+//        Integer legNumber = bg.getClientYacht().getLegNumber();
 //        List<Corner> markSequence = courseData.getMarkSequence();
 //
 //        if (legNumber == 0) {
@@ -352,10 +374,19 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 
     /**
      * Updates the wind direction arrow and text as from info from the StreamParser
+     * @param direction the from north angle of the wind.
      */
-    private void updateWindDirection() {
+    private void updateWindDirection(double direction) {
-        windDirectionText.setText(String.format("%.1f°", raceState.getWindDirection()));
+        windDirectionText.setText(String.format("%.1f°", direction));
-        windArrowText.setRotate(raceState.getWindDirection());
+        windArrowText.setRotate(direction);
+    }
+
+    /**
+     * Updates the speed of the wind as displayed by the info pane.
+     * @param windSpeed Windspeed in knots.
+     */
+    private void updateWindSpeed(double windSpeed) {
+        windSpeedText.setText("Speed: " + String.format("%.1f", windSpeed) + " Knots");
     }
 
 
@@ -375,26 +406,20 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
      * Updates the order of the yachts as from the StreamParser and sets them in the yacht order
      * section
      */
-    private void updateOrder() {
+    private void updateOrder(ObservableList<ClientYacht> yachts) {
-//        positionVbox.getChildren().removeAll();
-//        positionVbox.getStylesheets().add(getClass().getResource("/css/master.css").toString());
-
-        // list of racing yacht id
-        List<ClientYacht> sorted = new ArrayList<>(participants.values());
-        sorted.sort(Comparator.comparingInt(ClientYacht::getPositionInteger));
         List<Text> vboxEntries = new ArrayList<>();
 
-        for (ClientYacht clientYacht : sorted) {
+        for (int i = 0; i < yachts.size(); i++) {
 //            System.out.println("yacht == null  " + String.valueOf(yacht == null));
-            if (clientYacht.getBoatStatus() == 3) {  // 3 is finish status
+            if (yachts.get(i).getBoatStatus() == 3) {  // 3 is finish status
-                Text textToAdd = new Text(clientYacht.getPositionInteger() + ". " +
+                Text textToAdd = new Text(i + 1 + ". " +
-                    clientYacht.getShortName() + " (Finished)");
+                    yachts.get(i).getShortName() + " (Finished)");
                 textToAdd.setFill(Paint.valueOf("#d3d3d3"));
                 vboxEntries.add(textToAdd);
 
             } else {
-                Text textToAdd = new Text(clientYacht.getPositionInteger() + ". " +
+                Text textToAdd = new Text(i + 1 + ". " +
-                    clientYacht.getShortName() + " ");
+                    yachts.get(i).getShortName() + " ");
                 textToAdd.setFill(Paint.valueOf("#d3d3d3"));
                 textToAdd.setStyle("");
                 vboxEntries.add(textToAdd);
@@ -403,13 +428,6 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
         Platform.runLater(() ->
             positionVbox.getChildren().setAll(vboxEntries)
         );
-//            participants.forEach((id, yacht) ->{
-//                Text textToAdd = new Text(yacht.getPosition() + ". " +
-//                    yacht.getShortName() + " ");
-//                textToAdd.setFill(Paint.valueOf("#d3d3d3"));
-//                textToAdd.setStyle("");
-//                positionVbox.getChildren().add(textToAdd);
-//            });
     }
 
 
@@ -488,8 +506,7 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
     }
 
 
-    public Line makeLeftLayline(Point2D startPoint, Double layLineAngle, Double baseAngle) {
+    public Line  makeLeftLayline(Point2D startPoint, Double layLineAngle, Double baseAngle) {
-
         Point2D ep = getPointRotation(startPoint, 50.0, baseAngle + layLineAngle);
         Line line = new Line(startPoint.getX(), startPoint.getY(), ep.getX(), ep.getY());
         return line;
@@ -507,8 +524,8 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 
 
     /**
-     * Initialised the combo box with any yachts currently in the race and adds the required
+     * Initialised the combo box with any yachts currently in the race and adds the required listener
-     * listener for the combobox to take action upon selection
+     * for the combobox to take action upon selection
      */
     private void initialiseBoatSelectionComboBox() {
         yachtSelectionComboBox.setItems(
@@ -580,9 +597,9 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
     /**
      * Sets all the annotations of the selected yacht to be visible and all others to be hidden
      *
-     * @param clientYacht The yacht for which we want to view all annotations
+     * @param yacht The yacht for which we want to view all annotations
      */
-    private void setSelectedBoat(ClientYacht clientYacht) {
+    private void setSelectedBoat(ClientYacht yacht) {
 //        for (BoatObject bg : gameViewController.getBoatGroups()) {
 //            //We need to iterate over all race groups to get the matching yacht group belonging to this yacht if we
 //            //are to toggle its annotations, there is no other backwards knowledge of a yacht to its yachtgroup.
@@ -596,23 +613,8 @@ public class RaceViewController extends Thread implements ImportantAnnotationDel
 //        }
     }
 
-    public void updateRaceData(RaceXMLData raceData) {
+    public void updateRaceData (RaceXMLData raceData) {
         this.courseData = raceData;
         gameView.updateBorder(raceData.getCourseLimit());
     }
-
-    /**
-     * Called by game client after receiving yacht event packet. Parameter subject id is the
-     * offending yacht. This function in turn will pass the yacht location to game view to display a
-     * collision alert.
-     *
-     * @param subjectId source id of offending yacht
-     */
-    public void showCollision(Long subjectId) {
-        gameView.drawCollision(participants.get((int) (long) subjectId).getLocation());
-    }
-
-    public GameView getGameView() {
-        return gameView;
-    }
 }

src/main/java/seng302/visualiser/fxObjects/AnnotationBox.java

@@ -95,7 +95,7 @@ public class AnnotationBox extends Group {
         background.setStroke(theme);
         background.setStrokeWidth(2);
         background.setCache(true);
-        background.setCacheHint(CacheHint.SPEED);
+        background.setCacheHint(CacheHint.SCALE);
         this.getChildren().add(background);
     }
 
@@ -213,7 +213,7 @@ public class AnnotationBox extends Group {
         Text text = new Text();
         text.setFill(theme);
         text.setStrokeWidth(2);
-        text.setCacheHint(CacheHint.SPEED);
+//        text.setCacheHint(CacheHint.QUALITY);
         text.setCache(true);
         return text;
     }

src/main/java/seng302/visualiser/fxObjects/BoatObject.java

@@ -1,9 +1,9 @@
 package seng302.visualiser.fxObjects;
 
 import java.util.ArrayList;
+import java.util.List;
 import javafx.application.Platform;
 import javafx.geometry.Point2D;
-import javafx.scene.CacheHint;
 import javafx.scene.Group;
 import javafx.scene.Node;
 import javafx.scene.paint.Color;
@@ -11,6 +11,7 @@ import javafx.scene.paint.Paint;
 import javafx.scene.shape.Line;
 import javafx.scene.shape.Polygon;
 import javafx.scene.shape.Polyline;
+import javafx.scene.shape.StrokeLineCap;
 import javafx.scene.transform.Rotate;
 
 /**
@@ -23,6 +24,12 @@ import javafx.scene.transform.Rotate;
  */
 public class BoatObject extends Group {
 
+    @FunctionalInterface
+    public interface SelectedBoatListener {
+
+        void notifySelected(BoatObject boatObject, Boolean isSelected);
+    }
+
     //Constants for drawing
     private static final double BOAT_HEIGHT = 15d;
     private static final double BOAT_WIDTH = 10d;
@@ -41,9 +48,11 @@ public class BoatObject extends Group {
     private double distanceTravelled, lastRotation;
     private Point2D lastPoint;
     private Paint colour = Color.BLACK;
-    private Boolean isSelected, destinationSet;  //All boats are initialised as selected
+    private Boolean isSelected = false, destinationSet;  //All boats are initialised as selected
     private boolean isPlayer = false;
 
+    private List<SelectedBoatListener> selectedBoatListenerListeners = new ArrayList<>();
+
     /**
      * Creates a BoatGroup with the default triangular boat polygon.
      */
@@ -85,7 +94,7 @@ public class BoatObject extends Group {
         });
         boatPoly.setOnMouseClicked(event -> setIsSelected(!isSelected));
         boatPoly.setCache(true);
-        boatPoly.setCacheHint(CacheHint.SPEED);
+//        boatPoly.setCacheHint(CacheHint.SPEED);
 
 //        annotationBox = new AnnotationBox();
 //        annotationBox.setFill(colour);
@@ -287,6 +296,7 @@ public class BoatObject extends Group {
 //    }
 
     public void setIsSelected(Boolean isSelected) {
+        updateListener(isSelected);
         this.isSelected = isSelected;
         setLineGroupVisible(isSelected);
         setWakeVisible(isSelected);
@@ -352,7 +362,8 @@ public class BoatObject extends Group {
             BOAT_WIDTH / 1.75, BOAT_HEIGHT / 1.75
         );
         boatPoly.setStroke(Color.BLACK);
-        boatPoly.setStrokeWidth(3);
+        boatPoly.setStrokeWidth(2);
+        boatPoly.setStrokeLineCap(StrokeLineCap.ROUND);
         isPlayer = true;
         animateSail();
     }
@@ -365,6 +376,10 @@ public class BoatObject extends Group {
         lastHeading = heading;
     }
 
+    public Boolean getSelected() {
+        return isSelected;
+    }
+
     public void setTrajectory(double heading, double velocity, double scaleFactorX, double scaleFactorY) {
 //        wake.setRotation(lastHeading - heading, velocity);
 //        rotateTo(heading);
@@ -372,4 +387,14 @@ public class BoatObject extends Group {
 //        yVelocity = Math.sin(Math.toRadians(heading)) * velocity * scaleFactorY;
         lastHeading = heading;
     }
+
+    private void updateListener(Boolean isSelected) {
+        for (SelectedBoatListener sbl : selectedBoatListenerListeners) {
+            sbl.notifySelected(this, isSelected);
+        }
+    }
+
+    public void addSelectedBoatListener(SelectedBoatListener sbl) {
+        selectedBoatListenerListeners.add(sbl);
+    }
 }

src/main/java/seng302/visualiser/fxObjects/MarkArrowFactory.java

@@ -0,0 +1,147 @@
+package seng302.visualiser.fxObjects;
+
+import javafx.scene.Group;
+import javafx.scene.paint.Color;
+import javafx.scene.paint.Paint;
+import javafx.scene.shape.*;
+import javafx.scene.transform.Rotate;
+
+// TODO: 16/08/17 this class used to be well written... FeelsBadMan. Maybe lose the ternary operators.
+/**
+ * Factory class for making rounding arrows for mark objects out of JavaFX objects.
+ */
+public class MarkArrowFactory {
+
+    /**
+     * The side of the boat that will be closest to the mark.
+     */
+    public enum RoundingSide {
+        PORT,
+        STARBOARD,
+    }
+
+    public static final double MARK_ARROW_SEPARATION = 15;
+    public static final double ARROW_LENGTH = 75;
+    public static final double ARROW_HEAD_DEPTH = 10;
+    public static final double ARROW_HEAD_WIDTH = 6;
+    public static final double STROKE_WIDTH = 3;
+
+    /**
+     * Creates an entry arrow group showing an arrow into and out of the rounding area. It is centered on (0, 0).
+     * @param roundingSide The side of the boat that will be closest to the mark.
+     * @param angleOfEntry The angle between this mark and the last one as a heading from north in degrees.
+     * @param angleOfExit The angle between this mark and the next one as a heading from north in degrees.
+     * @param colour The desired colour of the arrows.
+     * @return The group containing all JavaFX objects.
+     */
+    public static Group constructEntryArrow (RoundingSide roundingSide, double angleOfEntry,
+                                             double angleOfExit, Paint colour) {
+        if (roundingSide == RoundingSide.PORT && angleOfEntry < angleOfExit && Math.abs(angleOfExit - angleOfEntry) < 180) {
+            return makeInteriorAngle(roundingSide, angleOfExit, angleOfEntry, colour);
+        } else if (roundingSide == RoundingSide.STARBOARD && angleOfEntry > angleOfExit && -Math.abs(angleOfEntry - angleOfExit) > -180) {
+            return makeInteriorAngle(roundingSide, angleOfExit, angleOfEntry, colour);
+        }
+
+        angleOfEntry = 180 - angleOfEntry;
+        Group arrow = new Group();
+        Group exitSection = constructExitArrow(roundingSide, angleOfExit, colour);
+        angleOfExit = 180 - angleOfExit;
+        Arc roundSection = new Arc(
+            0, 0, MARK_ARROW_SEPARATION, MARK_ARROW_SEPARATION,
+            (roundingSide == RoundingSide.PORT ? -180 : 0) + angleOfEntry,
+            roundingSide == RoundingSide.PORT ? Math.abs(angleOfExit - angleOfEntry) : -Math.abs(angleOfEntry - angleOfExit)
+        );
+        roundSection.setStrokeWidth(STROKE_WIDTH);
+        roundSection.setType(ArcType.OPEN);
+        roundSection.setStroke(colour);
+        roundSection.setFill(new Color(0,0,0,0));
+        Polygon entrySection = constructLineSegment(
+                roundingSide == RoundingSide.PORT ? RoundingSide.STARBOARD : RoundingSide.PORT, 180 + angleOfEntry, colour
+        );
+        arrow.getChildren().addAll(exitSection, roundSection, entrySection);
+        return arrow;
+    }
+
+    private static Group makeInteriorAngle (RoundingSide roundingSide, double angleOfExit, double angleOfEntry, Paint colour) {
+        Group arrow = new Group();
+        Polygon lineSegment;
+        angleOfEntry = Math.toRadians(360 - angleOfEntry);
+        angleOfExit = Math.toRadians(180 - angleOfExit);
+        int multiplier = roundingSide == RoundingSide.STARBOARD ? -1 : 1;
+        double xStart = multiplier * MARK_ARROW_SEPARATION * Math.sin(angleOfEntry + Math.PI / 2);
+        double yStart = multiplier * MARK_ARROW_SEPARATION * Math.cos(angleOfEntry + Math.PI / 2);
+        xStart = xStart + (ARROW_LENGTH * Math.sin(angleOfEntry));
+        yStart = yStart + (ARROW_LENGTH * Math.cos(angleOfEntry));
+        multiplier = roundingSide == RoundingSide.STARBOARD ? 1 : -1;
+        double xEnd = multiplier * MARK_ARROW_SEPARATION * Math.sin(angleOfExit + Math.PI / 2);
+        double yEnd = multiplier * MARK_ARROW_SEPARATION * Math.cos(angleOfExit + Math.PI / 2);
+        xEnd = xEnd + (ARROW_LENGTH * Math.sin(angleOfExit));
+        yEnd = yEnd + (ARROW_LENGTH * Math.cos(angleOfExit));
+        lineSegment = new Polygon(
+            xStart, yStart,
+            xEnd, yEnd
+        );
+        lineSegment.setStroke(colour);
+        lineSegment.setFill(Color.BLUE);
+        lineSegment.setStrokeWidth(STROKE_WIDTH);
+        lineSegment.setStrokeLineCap(StrokeLineCap.ROUND);
+        Polyline arrowHead = constructArrowHead(
+            90 + Math.toDegrees(Math.atan2(yStart - yEnd, xEnd - xStart)),
+            colour
+        );
+        arrowHead.setLayoutX(xEnd);
+        arrowHead.setLayoutY(yEnd);
+        arrow.getChildren().addAll(lineSegment, arrowHead);
+        return arrow;
+    }
+
+    /**
+     * Creates an exit arrow group pointing towards the next mark.
+     * @param roundingSide The side of the boat that will be closest to the mark.
+     * @param angle The angle to the next mark as a heading from north in degrees.
+     * @param colour The colour of the arrow.
+     * @return The group containing all the JavaFX objects.
+     */
+    public static Group constructExitArrow (RoundingSide roundingSide, double angle, Paint colour) {
+        angle = 180 - angle;
+        Group arrow = new Group();
+        Polygon arrowBody = constructLineSegment(roundingSide, angle, colour);
+        Polyline arrowHead = constructArrowHead(angle, colour);
+        arrowHead.setLayoutX(arrowBody.getPoints().get(2));
+        arrowHead.setLayoutY(arrowBody.getPoints().get(3));
+        arrow.getChildren().addAll(arrowBody, arrowHead);
+        return arrow;
+    }
+
+    private static Polygon constructLineSegment (RoundingSide roundingSide, double angle, Paint colour) {
+        Polygon lineSegment;
+        angle = Math.toRadians(angle);
+        int multiplier = roundingSide == RoundingSide.STARBOARD ? 1 : -1;
+        double xStart = multiplier * MARK_ARROW_SEPARATION * Math.sin(angle + Math.PI / 2);
+        double yStart = multiplier * MARK_ARROW_SEPARATION * Math.cos(angle + Math.PI / 2);
+        double xEnd = xStart + (ARROW_LENGTH * Math.sin(angle));
+        double yEnd = yStart + (ARROW_LENGTH * Math.cos(angle));
+        lineSegment = new Polygon(
+                xStart, yStart,
+                xEnd, yEnd
+        );
+        lineSegment.setStroke(colour);
+        lineSegment.setFill(Color.BLUE);
+        lineSegment.setStrokeWidth(STROKE_WIDTH);
+        lineSegment.setStrokeLineCap(StrokeLineCap.ROUND);
+        return lineSegment;
+    }
+
+    private static Polyline constructArrowHead (double rotation, Paint colour) {
+        Polyline arrow = new Polyline(
+            -ARROW_HEAD_WIDTH, -ARROW_HEAD_DEPTH,
+            0, 0,
+            ARROW_HEAD_WIDTH, -ARROW_HEAD_DEPTH
+        );
+        arrow.getTransforms().add(new Rotate(-rotation));
+        arrow.setStrokeLineCap(StrokeLineCap.ROUND);
+        arrow.setStroke(colour);
+        arrow.setStrokeWidth(STROKE_WIDTH);
+        return arrow;
+    }
+}

src/main/java/seng302/visualiser/fxObjects/Marker.java

@@ -1,19 +1,98 @@
 package seng302.visualiser.fxObjects;
 
+import java.util.ArrayList;
+import java.util.List;
+import javafx.application.Platform;
+import javafx.scene.Group;
+import javafx.scene.paint.Color;
 import javafx.scene.paint.Paint;
 import javafx.scene.shape.Circle;
 
 /**
- * Visual object for a mark.
+ * Visual object for a mark. Contains a coloured circle and any specified arrows.
  */
-public class Marker extends Circle {
+public class Marker extends Group {
 
+    private Circle mark = new Circle();
+    private Paint colour = Color.BLACK;
+    private List<Group> enterArrows = new ArrayList<>();
+    private List<Group> exitArrows = new ArrayList<>();
+    private int enterArrowIndex = 0;
+    private int exitArrowIndex = 0;
+
+    /**
+     * Creates a new Marker containing only a circle. The default colour is black.
+     */
     public Marker() {
-        super.setRadius(5);
+        mark.setRadius(5);
+        mark.setCenterX(0);
+        mark.setCenterY(0);
+        Platform.runLater(() -> this.getChildren().addAll(mark, new Group())); //Empty group placeholder or arrows.
     }
 
+    /**
+     * Creates a new Marker containing only a circle of the given colour.
+     * @param colour the desired colour for the marker.
+     */
     public Marker(Paint colour) {
         this();
-        super.setFill(colour);
+        this.colour = colour;
+        mark.setFill(colour);
+    }
+
+    /**
+     * Adds an exit and entry arrow pair to the mark. Arrows are hidden and shown in the order they
+     * are created by calling showNextEnterArrow() or showNextExitArrow()
+     * @param roundingSide the side the marker will be from the perspective of the arrow.
+     * @param entryAngle The angle the arrow will point towards a marker
+     * @param exitAngle The angle the arrow wil point from the marker.
+     */
+    public void addArrows(MarkArrowFactory.RoundingSide roundingSide, double entryAngle,
+        double exitAngle) {
+
+        enterArrows.add(
+            MarkArrowFactory.constructEntryArrow(roundingSide, entryAngle, exitAngle, colour)
+        );
+        exitArrows.add(
+            MarkArrowFactory.constructExitArrow(roundingSide, exitAngle, colour)
+        );
+//        Platform.runLater(() -> {
+//            this.getChildren().add(enterArrows.get(enterArrows.size()-1));
+//            this.getChildren().add(exitArrows.get(exitArrows.size()-1));
+//        });
+    }
+
+    /**
+     * Shows the next EnterArrow. Does nothing if there are no more enter arrows. Other arrows become hidden.
+     */
+    public void showNextEnterArrow() {
+        showArrow(enterArrows, enterArrowIndex);
+        enterArrowIndex++;
+    }
+
+    /**
+     * Shows the next ExitArrow. Does nothing if there are no more enter arrows. Other arrows become hidden.
+     */
+    public void showNextExitArrow() {
+        showArrow(exitArrows, exitArrowIndex);
+        exitArrowIndex++;
+    }
+
+    private void showArrow(List<Group> arrowList, int arrowListIndex) {
+        if (arrowListIndex < arrowList.size()) {
+            if (arrowListIndex == 1) {;
+            }
+            Platform.runLater(() -> {
+                this.getChildren().remove(1);
+                this.getChildren().add(arrowList.get(arrowListIndex));
+            });
+        }
+    }
+
+    /**
+     * Hides all arrows.
+     */
+    public void hideAllArrows() {
+        Platform.runLater(() -> this.getChildren().setAll(mark, new Group()));
     }
 }

src/main/resources/server_config/xml_templates/race.ftlh

@@ -24,8 +24,8 @@
             <Mark SeqID="2" Name="Lee Gate 2" TargetLat="57.671629" TargetLng="11.840951" SourceID="125" />
         </CompoundMark>
         <CompoundMark CompoundMarkID="4" Name="Mark3">
-            <Mark SeqID="1" Name="Wind Gate 1" TargetLat="57.665316" TargetLng="11.827184" SourceID="126" />
+            <Mark SeqID="1" Name="Wind Gate 1" TargetLat="57.664190" TargetLng="11.829576" SourceID="126" />
-            <Mark SeqID="2" Name="Wind Gate 2" TargetLat="57.664190" TargetLng="11.829576" SourceID="127" />
+            <Mark SeqID="2" Name="Wind Gate 2" TargetLat="57.665316" TargetLng="11.827184" SourceID="127" />
         </CompoundMark>
         <CompoundMark CompoundMarkID="5" Name="Mark4">
             <Mark SeqID="1" Name="Finish Line 1" TargetLat="57.672350" TargetLng="11.842535" SourceID="128" />
@@ -38,12 +38,7 @@
         <Corner SeqID="3" CompoundMarkID="3" Rounding="SP" ZoneSize="3" />
         <Corner SeqID="4" CompoundMarkID="4" Rounding="PS" ZoneSize="3" />
         <Corner SeqID="5" CompoundMarkID="3" Rounding="SP" ZoneSize="3" />
-        <Corner SeqID="6" CompoundMarkID="4" Rounding="PS" ZoneSize="3" />
+        <Corner SeqID="6" CompoundMarkID="5" Rounding="PS" ZoneSize="3" />
-        <Corner SeqID="7" CompoundMarkID="3" Rounding="SP" ZoneSize="3" />
-        <Corner SeqID="8" CompoundMarkID="4" Rounding="PS" ZoneSize="3" />
-        <Corner SeqID="9" CompoundMarkID="3" Rounding="SP" ZoneSize="3" />
-        <Corner SeqID="10" CompoundMarkID="4" Rounding="PS" ZoneSize="3" />
-        <Corner SeqID="11" CompoundMarkID="5" Rounding="PS" ZoneSize="3" />
     </CompoundMarkSequence>
     <CourseLimit>
         <Limit SeqID="1" Lat="57.6739450" Lon="11.8417100" />

src/main/resources/views/RaceView.fxml

@@ -1,5 +1,11 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
+<?import java.lang.*?>
+<?import javafx.scene.chart.*?>
+<?import javafx.scene.control.*?>
+<?import javafx.scene.layout.*?>
+<?import javafx.scene.shape.*?>
+<?import javafx.scene.text.*?>
 <?import javafx.scene.chart.CategoryAxis?>
 <?import javafx.scene.chart.LineChart?>
 <?import javafx.scene.chart.NumberAxis?>
@@ -17,40 +23,60 @@
 <?import javafx.scene.shape.Circle?>
 <?import javafx.scene.text.Font?>
 <?import javafx.scene.text.Text?>
-<GridPane prefHeight="960.0" prefWidth="1530.0" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0" xmlns="http://javafx.com/javafx/8" xmlns:fx="http://javafx.com/fxml/1" fx:controller="seng302.visualiser.controllers.RaceViewController">
+
-    <columnConstraints>
+<AnchorPane maxHeight="-Infinity" maxWidth="-Infinity" minHeight="-Infinity" minWidth="-Infinity" prefHeight="998.0" prefWidth="1530.0" xmlns="http://javafx.com/javafx/8" xmlns:fx="http://javafx.com/fxml/1" fx:controller="seng302.visualiser.controllers.RaceViewController">
-        <ColumnConstraints maxWidth="246.0" minWidth="246.0" prefWidth="246.0" />
+   <children>
-        <ColumnConstraints hgrow="SOMETIMES" minWidth="10.0" prefWidth="1034.0" />
+       <AnchorPane layoutX="322.0" layoutY="130.0" prefHeight="998.0" prefWidth="1281.0"
-    </columnConstraints>
+         style="-fx-background-color: skyblue;" AnchorPane.bottomAnchor="0.0"
-    <rowConstraints>
+         AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0">
-        <RowConstraints minHeight="500.0" prefHeight="30.0" vgrow="SOMETIMES" />
+           <children>
-        <RowConstraints />
+               <GridPane prefHeight="998.0" prefWidth="1281.0" AnchorPane.bottomAnchor="0.0"
-        <RowConstraints />
+                 AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0"
-    </rowConstraints>
+                 AnchorPane.topAnchor="0.0">
-    <children>
+                   <columnConstraints>
-        <AnchorPane prefHeight="960.0" prefWidth="250.0" style="-fx-background-color: #2C2c36;" GridPane.rowSpan="3">
+                       <ColumnConstraints hgrow="SOMETIMES" maxWidth="630.0" minWidth="10.0"
-            <children>
+                         prefWidth="68.0"/>
-                <Label layoutX="11.0" layoutY="283.0" text="Team Position" textFill="WHITE" />
+                       <ColumnConstraints hgrow="SOMETIMES" maxWidth="1213.0" minWidth="10.0"
-                <Label layoutX="13.0" layoutY="432.0" text="Settings" textFill="WHITE" />
+                         prefWidth="1213.0"/>
-                <Label layoutX="11.0" layoutY="41.0" text="Timer" textFill="WHITE" />
+                   </columnConstraints>
-                <Label layoutX="11.0" layoutY="112.0" text="Wind direction" textFill="WHITE" />
+                   <rowConstraints>
-                <Circle fx:id="windBackgroundCircle" blendMode="DARKEN" fill="#3dcdc8" layoutX="110.0" layoutY="190.0" radius="35.0" stroke="#d7d7d7" strokeType="INSIDE" strokeWidth="3.0" />
+                       <RowConstraints maxHeight="489.0" minHeight="1.0" prefHeight="24.0"
-                <Text fx:id="windArrowText" fill="#a8a8a8" layoutX="86.0" layoutY="210.0" strokeType="OUTSIDE" strokeWidth="0.0" text="↓">
+                         vgrow="SOMETIMES"/>
-                    <font>
+                       <RowConstraints maxHeight="997.0" minHeight="10.0" prefHeight="974.0"
-                        <Font name="AdobeArabic-Regular" size="55.0" />
+                         vgrow="SOMETIMES"/>
-                    </font>
+                   </rowConstraints>
-                </Text>
+                   <children>
-                <Text fx:id="windDirectionText" fill="#d3d3d3" layoutX="171.0" layoutY="238.0" strokeType="OUTSIDE" strokeWidth="0.0" text="0.0°" textAlignment="RIGHT">
+                       <AnchorPane fx:id="contentAnchorPane" prefHeight="200.0" prefWidth="200.0"
-                    <font>
+                         GridPane.columnSpan="2" GridPane.rowSpan="2"/>
-                        <Font name="System Bold" size="13.0" />
+                       <Text fx:id="fpsDisplay" strokeType="OUTSIDE" strokeWidth="0.0" text="60 FPS"
-                    </font>
+                         GridPane.halignment="CENTER" GridPane.valignment="CENTER"/>
-                </Text>
+                   </children>
+               </GridPane>
+           </children>
+       </AnchorPane>
+      <AnchorPane prefHeight="960.0" prefWidth="250.0" style="-fx-background-color: #2C2c36;" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.topAnchor="0.0">
+         <children>
+            <Label layoutX="11.0" layoutY="283.0" text="Team Position" textFill="WHITE" />
+            <Label layoutX="13.0" layoutY="432.0" text="Settings" textFill="WHITE" />
+            <Label layoutX="11.0" layoutY="41.0" text="Timer" textFill="WHITE" />
+            <Label layoutX="11.0" layoutY="112.0" text="Wind direction" textFill="WHITE" />
+            <Circle fx:id="windBackgroundCircle" blendMode="DARKEN" fill="#3dcdc8" layoutX="110.0" layoutY="190.0" radius="35.0" stroke="#d7d7d7" strokeType="INSIDE" strokeWidth="3.0" />
+            <Text fx:id="windArrowText" fill="#a8a8a8" layoutX="86.0" layoutY="210.0" strokeType="OUTSIDE" strokeWidth="0.0" text="↓">
+               <font>
+                  <Font name="AdobeArabic-Regular" size="55.0" />
+               </font>
+            </Text>
+            <Text fx:id="windDirectionText" fill="#d3d3d3" layoutX="171.0" layoutY="238.0" strokeType="OUTSIDE" strokeWidth="0.0" text="0.0°" textAlignment="RIGHT">
+               <font>
+                  <Font name="System Bold" size="13.0" />
+               </font>
+            </Text>
             <Text fx:id="windSpeedText" fill="#d3d3d3" layoutX="12.0" layoutY="237.0" strokeType="OUTSIDE" strokeWidth="0.0" text="0.0 Knot" textAlignment="RIGHT">
                <font>
                   <Font name="System Bold" size="13.0" />
                </font>
             </Text>
-                <CheckBox fx:id="toggleFps" focusTraversable="false" graphicTextGap="0.0" layoutX="21.0" layoutY="453.0" mnemonicParsing="false" prefHeight="18.0" prefWidth="143.0" selected="true" styleClass="ui-checkbox" text="Show FPS" textFill="WHITE" />
+            <CheckBox fx:id="toggleFps" focusTraversable="false" graphicTextGap="0.0" layoutX="21.0" layoutY="453.0" mnemonicParsing="false" prefHeight="18.0" prefWidth="143.0" selected="true" styleClass="ui-checkbox" text="Show FPS" textFill="WHITE" />
             <VBox fx:id="positionVbox" layoutX="12.0" layoutY="304.0" prefHeight="116.0" prefWidth="200.0" styleClass="text-white" />
             <Pane layoutX="11.0" layoutY="39.0" maxHeight="-Infinity" maxWidth="-Infinity" minHeight="-Infinity" minWidth="-Infinity" prefHeight="51.0" prefWidth="193.0">
                <children>
@@ -67,17 +93,14 @@
             <Text fill="WHITE" layoutX="11.0" layoutY="649.0" strokeType="OUTSIDE" strokeWidth="0.0" text="Boat Selection" />
             <ComboBox fx:id="yachtSelectionComboBox" focusTraversable="false" layoutX="37.0" layoutY="664.0" prefHeight="25.0" prefWidth="170.0" promptText="Select Yacht" styleClass="combo-box-base" />
             <LineChart fx:id="raceSparkLine" layoutX="-1.0" layoutY="719.0" legendVisible="false" prefHeight="277.0" prefWidth="246.0" title="Boat Positions">
-              <xAxis>
+               <xAxis>
-                <CategoryAxis label="Leg Number" side="BOTTOM" styleClass="spark-line-xaxis" />
+                  <CategoryAxis label="Leg Number" side="BOTTOM" styleClass="spark-line-xaxis" />
-              </xAxis>
+               </xAxis>
-              <yAxis>
+               <yAxis>
-                <NumberAxis fx:id="sparklineYAxis" minorTickCount="1" minorTickLength="1.0" side="LEFT" styleClass="spark-line-yaxis" tickLabelGap="1.0" tickUnit="1.0" upperBound="7.0" />
+                  <NumberAxis fx:id="sparklineYAxis" minorTickCount="1" minorTickLength="1.0" side="LEFT" styleClass="spark-line-yaxis" tickLabelGap="1.0" tickUnit="1.0" upperBound="7.0" />
-              </yAxis>
+               </yAxis>
             </LineChart>
-            </children>
+         </children>
-        </AnchorPane>
+      </AnchorPane>
-        <AnchorPane fx:id="contentAnchorPane" style="-fx-background-color: skyblue;" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0" GridPane.columnIndex="1" GridPane.halignment="LEFT" GridPane.rowSpan="2147483647" GridPane.valignment="TOP">
+   </children>
-
+</AnchorPane>
-        </AnchorPane>
-    </children>
-</GridPane>

src/test/java/seng302/visualiser/ClientToServerTests/RegularPacketsTest.java

@@ -1,9 +1,15 @@
 package seng302.visualiser.ClientToServerTests;
 
+import java.util.ArrayList;
 import org.junit.After;
+import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
+import seng302.gameServer.GameStages;
+import seng302.gameServer.GameState;
 import seng302.gameServer.MainServerThread;
+import seng302.gameServer.messages.BoatAction;
+import seng302.model.ServerYacht;
 import seng302.visualiser.ClientToServerThread;
 
 /**
@@ -16,39 +22,44 @@ public class RegularPacketsTest {
 
     @Before
     public void setup() throws Exception {
-//        new GameState("localhost");
+        new GameState("localhost");
-//        serverThread = new MainServerThread();
+        serverThread = new MainServerThread();
-//        clientThread = new ClientToServerThread("localhost", 4942);
+        clientThread = new ClientToServerThread("localhost", 4942);
-//        GameState.setCurrentStage(GameStages.RACING);
+        GameState.setCurrentStage(GameStages.RACING);
     }
 
     @Test
-    public void packetsSentAtRegularIntervals () throws Exception {
+    public void packetsSentAtRegularIntervals() {
-//        final double TEST_DISTANCE = 10.0;
+        try {
-//        serverThread.startGame();
+            final double TEST_DISTANCE = 10.0;
-//        SleepThreadMaxDelay();
+            serverThread.startGame();
-//        ServerYacht yacht = new ArrayList<>(GameState.getYachts().values()).get(0);
+            SleepThreadMaxDelay();
-//        double startAngle = yacht.getHeading();
+            ServerYacht yacht = new ArrayList<>(GameState.getYachts().values()).get(0);
-//        long startTime = System.currentTimeMillis();
+            double startAngle = yacht.getHeading();
-//        clientThread.sendBoatAction(BoatAction.UPWIND);
+            long startTime = System.currentTimeMillis();
-//        Thread.sleep(200);
+            clientThread.sendBoatAction(BoatAction.UPWIND); //start sending
-//        while (Math.abs(yacht.getHeading() - startAngle) < TEST_DISTANCE) {
+            Thread.sleep(200);
-//            Thread.sleep(1);
+            while (Math.abs(yacht.getHeading() - startAngle) < TEST_DISTANCE) {
-//        }
+                Thread.sleep(1);
-//        clientThread.sendBoatAction(BoatAction.MAINTAIN_HEADING);
+            }
-//        long endTime = System.currentTimeMillis();
+            clientThread.sendBoatAction(BoatAction.MAINTAIN_HEADING); //stop sending
-//        SleepThreadMaxDelay();
+            long endTime = System.currentTimeMillis();
-//        //Allowed to be two loops of delay due to loop delay and processing delay at client + server ends.
+            SleepThreadMaxDelay();
-//        Assert.assertEquals(
+            //Allowed to be two loops of delay due to loop delay and processing delay at client + server ends.
-//            TEST_DISTANCE / ServerYacht.TURN_STEP * ClientToServerThread.PACKET_SENDING_INTERVAL_MS,
+            Assert.assertEquals(
-//            (endTime - startTime), 2 * ClientToServerThread.PACKET_SENDING_INTERVAL_MS);
+                TEST_DISTANCE / ServerYacht.TURN_STEP
+                    * ClientToServerThread.PACKET_SENDING_INTERVAL_MS,
+                (endTime - startTime), 2 * ClientToServerThread.PACKET_SENDING_INTERVAL_MS);
+        } catch (Exception e) {
+            System.out.println("Caught expected exception.");
+        }
     }
 
 //    @Test
 //    public void testArbitraryPacketSent() throws Exception {
 //        serverThread.startGame();
 //        SleepThreadMaxDelay();
-//        Yacht yacht = new ArrayList<>(GameState.getYachts().values()).get(0);
+//        ServerYacht yacht = new ArrayList<>(GameState.getYachts().values()).get(0);
 //        boolean startState = yacht.getSailIn();
 //        clientThread.sendBoatAction(BoatAction.SAILS_IN);
 //        SleepThreadMaxDelay();
@@ -61,15 +72,17 @@ public class RegularPacketsTest {
      * @throws Exception Thrown if thread crashes or something
      */
     private void SleepThreadMaxDelay() throws Exception {
-        Thread.sleep(200);
+        Thread.sleep(100);
     }
 
     @After
-    public void teardown () throws Exception {
+    public void teardown() {
-//        clientThread.setSocketToClose();
+        try {
-//        serverThread.terminate();
+            clientThread.setSocketToClose();
-//        GameState.setCurrentStage(GameStages.LOBBYING);
+            serverThread.terminate();
-//        for (int i = 0; i<20; i++)
+            GameState.setCurrentStage(GameStages.LOBBYING);
-//            SleepThreadMaxDelay(); //Make sure socket is closed and toolkit remade.
+        } catch (Exception e) {
+            System.out.println("Caught expected exception.");
+        }
     }
 }

src/test/java/seng302/visualiser/map/BoatSailAnimationToggleTest.java

@@ -21,9 +21,9 @@ public class BoatSailAnimationToggleTest {
 
     @Test
     public void sailToggleTest() throws Exception {
-        assertFalse(yacht.getSailIn());
-        yacht.toggleSail();
         assertTrue(yacht.getSailIn());
+        yacht.toggleSail();
+        assertFalse(yacht.getSailIn());
     }
 
 }