X-Git-Url: http://gitweb.fperrin.net/?a=blobdiff_plain;f=tim%2Fprune%2Ffunction%2Fcharts%2FCharter.java;h=0bb377fe2b1c4fe728b1020c1fe6799456ce6a66;hb=4d5796d02a15808311c09448d79e6e7d1de9d636;hp=a3348c8ce0e35e8da11c4aa39062b9b18abdab28;hpb=54b9d8bc8f0025ccf97a67d9dd217ef1f9cf082f;p=GpsPrune.git

diff --git a/tim/prune/function/charts/Charter.java b/tim/prune/function/charts/Charter.java
index a3348c8..0bb377f 100644
--- a/tim/prune/function/charts/Charter.java
+++ b/tim/prune/function/charts/Charter.java
@@ -25,17 +25,17 @@ import javax.swing.JTextField;
 import javax.swing.SwingConstants;
 
 import tim.prune.App;
-import tim.prune.Config;
 import tim.prune.ExternalTools;
 import tim.prune.GenericFunction;
 import tim.prune.I18nManager;
-import tim.prune.data.Altitude;
+import tim.prune.config.Config;
 import tim.prune.data.DataPoint;
 import tim.prune.data.Distance;
 import tim.prune.data.Field;
 import tim.prune.data.Timestamp;
 import tim.prune.data.Track;
-import tim.prune.data.Distance.Units;
+import tim.prune.gui.profile.SpeedData;
+import tim.prune.gui.profile.VerticalSpeedData;
 import tim.prune.load.GenericFileFilter;
 
 /**
@@ -87,6 +87,12 @@ public class Charter extends GenericFunction
 	 */
 	public void begin()
 	{
+		// First check if gnuplot is available
+		if (!ExternalTools.isToolInstalled(ExternalTools.TOOL_GNUPLOT))
+		{
+			_app.showErrorMessage(getNameKey(), "dialog.charts.gnuplotnotfound");
+			return;
+		}
 		// Make dialog window
 		if (_dialog == null)
 		{
@@ -183,14 +189,15 @@ public class Charter extends GenericFunction
 		// button panel on bottom
 		JPanel buttonPanel = new JPanel();
 		buttonPanel.setLayout(new FlowLayout(FlowLayout.RIGHT));
-		// Gnuplot button
-		JButton gnuplotButton = new JButton(I18nManager.getText("button.gnuplotpath"));
-		gnuplotButton.addActionListener(new ActionListener() {
+		// ok button
+		JButton okButton = new JButton(I18nManager.getText("button.ok"));
+		okButton.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e) {
-				setGnuplotPath();
+				showChart(_app.getTrackInfo().getTrack());
+				_dialog.setVisible(false);
 			}
 		});
-		buttonPanel.add(gnuplotButton);
+		buttonPanel.add(okButton);
 		// Cancel button
 		JButton cancelButton = new JButton(I18nManager.getText("button.cancel"));
 		cancelButton.addActionListener(new ActionListener() {
@@ -199,15 +206,6 @@ public class Charter extends GenericFunction
 			}
 		});
 		buttonPanel.add(cancelButton);
-		// ok button
-		JButton okButton = new JButton(I18nManager.getText("button.ok"));
-		okButton.addActionListener(new ActionListener() {
-			public void actionPerformed(ActionEvent e) {
-				showChart(_app.getTrackInfo().getTrack());
-				_dialog.setVisible(false);
-			}
-		});
-		buttonPanel.add(okButton);
 		dialogPanel.add(buttonPanel, BorderLayout.SOUTH);
 		return dialogPanel;
 	}
@@ -221,7 +219,7 @@ public class Charter extends GenericFunction
 	private boolean setupDialog(Track inTrack)
 	{
 		boolean hasTimes = inTrack.hasData(Field.TIMESTAMP);
-		boolean hasAltitudes = inTrack.getAltitudeRange().hasRange();
+		boolean hasAltitudes = inTrack.hasAltitudeData();
 		_timeRadio.setEnabled(hasTimes);
 
 		// Add checks to prevent choosing unavailable combinations
@@ -277,7 +275,8 @@ public class Charter extends GenericFunction
 		OutputStreamWriter writer = null;
 		try
 		{
-			Process process = Runtime.getRuntime().exec(Config.getGnuplotPath() + " -persist");
+			final String gnuplotPath = Config.getConfigString(Config.KEY_GNUPLOT_PATH);
+			Process process = Runtime.getRuntime().exec(gnuplotPath + " -persist");
 			writer = new OutputStreamWriter(process.getOutputStream());
 			if (showSvg)
 			{
@@ -381,7 +380,7 @@ public class Charter extends GenericFunction
 		FileWriter tempFileWriter = null;
 		try {
 			tempFileWriter = new FileWriter(tempFile);
-			tempFileWriter.write("# Temporary data file for Prune charts\n\n");
+			tempFileWriter.write("# Temporary data file for GpsPrune charts\n\n");
 			for (int i=0; i<inTrack.getNumPoints(); i++) {
 				if (xValues.hasData(i) && yValues.hasData(i)) {
 					tempFileWriter.write("" + xValues.getData(i) + ", " + yValues.getData(i) + "\n");
@@ -398,9 +397,15 @@ public class Charter extends GenericFunction
 			catch (Exception e) {}
 		}
 
+		// Sort out units to use
+		final String distLabel = I18nManager.getText(Config.getUnitSet().getDistanceUnit().getShortnameKey());
+		final String altLabel  = I18nManager.getText(Config.getUnitSet().getAltitudeUnit().getShortnameKey());
+		final String speedLabel = I18nManager.getText(Config.getUnitSet().getSpeedUnit().getShortnameKey());
+		final String vertSpeedLabel = I18nManager.getText(Config.getUnitSet().getVerticalSpeedUnit().getShortnameKey());
+
 		// Set x axis label
 		if (inDistance) {
-			inWriter.write("set xlabel '" + I18nManager.getText("fieldname.distance") + " (" + getUnitsLabel("units.kilometres.short", "units.miles.short") + ")'\n");
+			inWriter.write("set xlabel '" + I18nManager.getText("fieldname.distance") + " (" + distLabel + ")'\n");
 		}
 		else {
 			inWriter.write("set xlabel '" + I18nManager.getText("fieldname.time") + " (" + I18nManager.getText("units.hours") + ")'\n");
@@ -411,19 +416,19 @@ public class Charter extends GenericFunction
 		switch (inYaxis)
 		{
 		case 0: // y axis is distance
-			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.distance") + " (" + getUnitsLabel("units.kilometres.short", "units.miles.short") + ")'\n");
+			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.distance") + " (" + distLabel + ")'\n");
 			chartTitle = I18nManager.getText("fieldname.distance");
 			break;
 		case 1: // y axis is altitude
-			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.altitude") + " (" + getUnitsLabel("units.metres.short", "units.feet.short") + ")'\n");
+			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.altitude") + " (" + altLabel + ")'\n");
 			chartTitle = I18nManager.getText("fieldname.altitude");
 			break;
 		case 2: // y axis is speed
-			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.speed") + " (" + getUnitsLabel("units.kmh", "units.mph") + ")'\n");
+			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.speed") + " (" + speedLabel + ")'\n");
 			chartTitle = I18nManager.getText("fieldname.speed");
 			break;
 		case 3: // y axis is vertical speed
-			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.verticalspeed") + " (" + getUnitsLabel("units.metrespersec", "units.feetpersec") + ")'\n");
+			inWriter.write("set ylabel '" + I18nManager.getText("fieldname.verticalspeed") + " (" + vertSpeedLabel + ")'\n");
 			chartTitle = I18nManager.getText("fieldname.verticalspeed");
 			break;
 		}
@@ -431,18 +436,6 @@ public class Charter extends GenericFunction
 		inWriter.write("plot '" + tempFile.getAbsolutePath() + "' title '" + chartTitle + "' with filledcurve y1=0 lt rgb \"#009000\"\n");
 	}
 
-	/**
-	 * Get the units label for the given keys
-	 * @param inMetric key if metric
-	 * @param inImperial key if imperial
-	 * @return display label with appropriate text
-	 */
-	private static String getUnitsLabel(String inMetric, String inImperial)
-	{
-		String key = Config.getUseMetricUnits()?inMetric:inImperial;
-		return I18nManager.getText(key);
-	}
-
 
 	/**
 	 * Calculate the distance values for each point in the given track
@@ -462,11 +455,10 @@ public class Charter extends GenericFunction
 			{
 				totalRads += DataPoint.calculateRadiansBetween(prevPoint, currPoint);
 			}
-			if (Config.getUseMetricUnits()) {
-				values.setData(i, Distance.convertRadiansToDistance(totalRads, Units.KILOMETRES));
-			} else {
-				values.setData(i, Distance.convertRadiansToDistance(totalRads, Units.MILES));
-			}
+
+			// distance values use currently configured units
+			values.setData(i, Distance.convertRadiansToDistance(totalRads));
+
 			prevPoint = currPoint;
 		}
 		return values;
@@ -507,10 +499,10 @@ public class Charter extends GenericFunction
 	private static ChartSeries getAltitudeValues(Track inTrack)
 	{
 		ChartSeries values = new ChartSeries(inTrack.getNumPoints());
-		Altitude.Format altFormat = Config.getUseMetricUnits()?Altitude.Format.METRES:Altitude.Format.FEET;
+		final double multFactor = Config.getUnitSet().getAltitudeUnit().getMultFactorFromStd();
 		for (int i=0; i<inTrack.getNumPoints(); i++) {
 			if (inTrack.getPoint(i).hasAltitude()) {
-				values.setData(i, inTrack.getPoint(i).getAltitude().getValue(altFormat));
+				values.setData(i, inTrack.getPoint(i).getAltitude().getMetricValue() * multFactor);
 			}
 		}
 		return values;
@@ -523,30 +515,17 @@ public class Charter extends GenericFunction
 	 */
 	private static ChartSeries getSpeedValues(Track inTrack)
 	{
-		// Calculate speeds and fill in in values array
-		ChartSeries values = new ChartSeries(inTrack.getNumPoints());
-		DataPoint prevPoint = null, currPoint = null, nextPoint = null;
-		DataPoint[] points = getDataPoints(inTrack, false);
+		// Calculate speeds using the same formula as the profile chart
+		SpeedData speeds = new SpeedData(inTrack);
+
+		final int numPoints = inTrack.getNumPoints();
+		ChartSeries values = new ChartSeries(numPoints);
 		// Loop over collected points
-		for (int i=1; i<(points.length-1); i++)
+		for (int i=0; i<numPoints; i++)
 		{
-			prevPoint = points[i-1];
-			currPoint = points[i];
-			nextPoint = points[i+1];
-			if (prevPoint != null && currPoint != null && nextPoint != null
-				&& nextPoint.getTimestamp().isAfter(currPoint.getTimestamp())
-				&& currPoint.getTimestamp().isAfter(prevPoint.getTimestamp()))
+			if (speeds.hasData(i))
 			{
-				// Calculate average speed between prevPoint and nextPoint
-				double rads = DataPoint.calculateRadiansBetween(prevPoint, currPoint)
-					+ DataPoint.calculateRadiansBetween(currPoint, nextPoint);
-				double time = nextPoint.getTimestamp().getSecondsSince(prevPoint.getTimestamp()) / 60.0 / 60.0;
-				// Convert to distance and pass to chartseries
-				if (Config.getUseMetricUnits()) {
-					values.setData(i, Distance.convertRadiansToDistance(rads, Units.KILOMETRES) / time);
-				} else {
-					values.setData(i, Distance.convertRadiansToDistance(rads, Units.MILES) / time);
-				}
+				values.setData(i, speeds.getData(i));
 			}
 		}
 		return values;
@@ -559,62 +538,26 @@ public class Charter extends GenericFunction
 	 */
 	private static ChartSeries getVertSpeedValues(Track inTrack)
 	{
-		// Calculate speeds and fill in in values array
-		ChartSeries values = new ChartSeries(inTrack.getNumPoints());
-		Altitude.Format altFormat = Config.getUseMetricUnits()?Altitude.Format.METRES:Altitude.Format.FEET;
-		DataPoint prevPoint = null, currPoint = null, nextPoint = null;
-		DataPoint[] points = getDataPoints(inTrack, true); // require that points have altitudes too
+		// Calculate speeds using the same formula as the profile chart
+		VerticalSpeedData speeds = new VerticalSpeedData(inTrack);
+
+		final int numPoints = inTrack.getNumPoints();
+		ChartSeries values = new ChartSeries(numPoints);
 		// Loop over collected points
-		for (int i=1; i<(points.length-1); i++)
+		for (int i=0; i<numPoints; i++)
 		{
-			prevPoint = points[i-1];
-			currPoint = points[i];
-			nextPoint = points[i+1];
-			if (prevPoint != null && currPoint != null && nextPoint != null
-				&& nextPoint.getTimestamp().isAfter(currPoint.getTimestamp())
-				&& currPoint.getTimestamp().isAfter(prevPoint.getTimestamp()))
+			if (speeds.hasData(i))
 			{
-				// Calculate average vertical speed between prevPoint and nextPoint
-				double vspeed = (nextPoint.getAltitude().getValue(altFormat) - prevPoint.getAltitude().getValue(altFormat))
-				 * 1.0 / nextPoint.getTimestamp().getSecondsSince(prevPoint.getTimestamp());
-				values.setData(i, vspeed);
+				values.setData(i, speeds.getData(i));
 			}
 		}
 		return values;
 	}
 
 
-	/**
-	 * Get an array of DataPoints with data for the charts
-	 * @param inTrack track object containing points
-	 * @param inRequireAltitudes true if only points with altitudes are considered
-	 * @return array of points with contiguous non-null elements (<= size) with timestamps
-	 */
-	private static DataPoint[] getDataPoints(Track inTrack, boolean inRequireAltitudes)
-	{
-		DataPoint[] points = new DataPoint[inTrack.getNumPoints()];
-		DataPoint currPoint = null;
-		int pointNum = 0;
-		// Loop over all points
-		for (int i=0; i<inTrack.getNumPoints(); i++)
-		{
-			currPoint = inTrack.getPoint(i);
-			if (currPoint != null && !currPoint.isWaypoint() && currPoint.hasTimestamp()
-				&& (!inRequireAltitudes || currPoint.hasAltitude()))
-			{
-				points[pointNum] = currPoint;
-				pointNum++;
-			}
-		}
-		// Any elements at the end of the array will stay null
-		// Also note, chronological order is not checked
-		return points;
-	}
-
-
 	/**
 	 * Select a file to write for the SVG output
-	 * @return
+	 * @return selected File object or null if cancelled
 	 */
 	private File selectSvgFile()
 	{
@@ -625,8 +568,8 @@ public class Charter extends GenericFunction
 			_fileChooser.setFileFilter(new GenericFileFilter("filetype.svg", new String[] {"svg"}));
 			_fileChooser.setAcceptAllFileFilterUsed(false);
 			// start from directory in config which should be set
-			File configDir = Config.getWorkingDirectory();
-			if (configDir != null) {_fileChooser.setCurrentDirectory(configDir);}
+			String configDir = Config.getConfigString(Config.KEY_TRACK_DIR);
+			if (configDir != null) {_fileChooser.setCurrentDirectory(new File(configDir));}
 		}
 		boolean chooseAgain = true;
 		while (chooseAgain)
@@ -650,9 +593,7 @@ public class Charter extends GenericFunction
 				{
 					return file;
 				}
-				else {
-					chooseAgain = true;
-				}
+				chooseAgain = true;
 			}
 		}
 		// Cancel pressed so no file selected
@@ -671,27 +612,4 @@ public class Charter extends GenericFunction
 		if (inNumCharts == 3) {return new int[] {40, 60, 20, 20, 0, 20};}
 		return new int[] {54, 46, 36, 18, 18, 18, 0, 18};
 	}
-
-	/**
-	 * Prompt the user to set/edit the path to gnuplot
-	 */
-	private void setGnuplotPath()
-	{
-		String currPath = Config.getGnuplotPath();
-		Object path = JOptionPane.showInputDialog(_dialog,
-			I18nManager.getText("dialog.charts.gnuplotpath"),
-			I18nManager.getText(getNameKey()),
-			JOptionPane.QUESTION_MESSAGE, null, null, "" + currPath);
-		if (path != null)
-		{
-			String pathString = path.toString().trim();
-			if (!pathString.equals("") && !pathString.equals(currPath)) {
-				Config.setGnuplotPath(pathString);
-				// warn if gnuplot still not found
-				if (!ExternalTools.isGnuplotInstalled()) {
-					_app.showErrorMessage(getNameKey(), "dialog.charts.gnuplotnotfound");
-				}
-			}
-		}
-	}
 }