Merge branch 'master' into support_for_philips_hue

ntim · ntim · commit ee3816fde708 · 2014-05-07T07:44:11.000+02:00
diff --git a/bin/install_hyperion.sh b/bin/install_hyperion.sh
@@ -34,13 +34,13 @@ fi
 echo 'Downloading hyperion'
 if [ $IS_OPENELEC -eq 1 ]; then
 	# OpenELEC has a readonly file system. Use alternative location
-	curl --get https://raw.github.com/tvdzwan/hyperion/master/deploy/hyperion.tar.gz | tar -C /storage -xz
-	curl --get https://raw.github.com/tvdzwan/hyperion/master/deploy/hyperion.deps.openelec-rpi.tar.gz | tar -C /storage/hyperion/bin -xz
+	curl -L --get https://raw.githubusercontent.com/tvdzwan/hyperion/master/deploy/hyperion.tar.gz | tar -C /storage -xz
+	curl -L --get https://raw.githubusercontent.com/tvdzwan/hyperion/master/deploy/hyperion.deps.openelec-rpi.tar.gz | tar -C /storage/hyperion/bin -xz
 
 	# modify the default config to have a correct effect path
 	sed -i 's:/opt:/storage:g' /storage/hyperion/config/hyperion.config.json
 else
-	wget https://raw.github.com/tvdzwan/hyperion/master/deploy/hyperion.tar.gz -O - | tar -C /opt -xz
+	wget https://raw.githubusercontent.com/tvdzwan/hyperion/master/deploy/hyperion.tar.gz -O - | tar -C /opt -xz
 fi
 
 # create links to the binaries
@@ -68,9 +68,9 @@ fi
 if [ $USE_INITCTL -eq 1 ]; then
 	echo 'Installing initctl script'
 	if [ $IS_RASPBMC -eq 1 ]; then
-		wget -N https://raw.github.com/tvdzwan/hyperion/master/deploy/hyperion.conf -P /etc/init/
+		wget -N https://raw.githubusercontent.com/tvdzwan/hyperion/master/deploy/hyperion.conf -P /etc/init/
 	else
-		wget -N https://raw.github.com/tvdzwan/hyperion/master/deploy/hyperion.xbian.conf -O /etc/init/hyperion.conf
+		wget -N https://raw.githubusercontent.com/tvdzwan/hyperion/master/deploy/hyperion.xbian.conf -O /etc/init/hyperion.conf
 	fi
 elif [ $USE_SERVICE -eq 1 ]; then
 	echo 'Installing startup script in init.d'
diff --git a/deploy/hyperion.tar.gz b/deploy/hyperion.tar.gz
diff --git a/include/hyperion/ImageProcessor.h b/include/hyperion/ImageProcessor.h
@@ -37,6 +37,9 @@ class ImageProcessor
 	///
 	void setSize(const unsigned width, const unsigned height);
 
+	/// Enable or disable the black border detector
+	void enableBalckBorderDetector(bool enable);
+
 	///
 	/// Processes the image to a list of led colors. This will update the size of the buffer-image
 	/// if required and call the image-to-leds mapping to determine the mean color per led.
@@ -142,7 +145,7 @@ class ImageProcessor
 	const LedString _ledString;
 
 	/// Flag the enables(true)/disabled(false) blackborder detector
-	const bool _enableBlackBorderRemoval;
+	bool _enableBlackBorderRemoval;
 
 	/// The processor for black border detection
 	hyperion::BlackBorderProcessor * _borderProcessor;
diff --git a/include/utils/Sleep.h b/include/utils/Sleep.h
@@ -0,0 +1,10 @@
+#pragma once
+
+#include <QThread>
+
+class Sleep : protected QThread {
+public:
+   static inline void msleep(unsigned long msecs) {
+	   QThread::msleep(msecs);
+   }
+};
diff --git a/include/xbmcvideochecker/XBMCVideoChecker.h b/include/xbmcvideochecker/XBMCVideoChecker.h
@@ -38,7 +38,7 @@ Q_OBJECT
 	/// @param grabPhoto Whether or not to grab when the XBMC photo player is playing
 	/// @param grabAudio Whether or not to grab when the XBMC audio player is playing
 	/// @param grabMenu Whether or not to grab when nothing is playing (in XBMC menu)
-	/// @param grabScreensaver 	Whether or not to grab when the XBMC screensaver is activated
+	/// @param grabScreensaver Whether or not to grab when the XBMC screensaver is activated
 	/// @param enable3DDetection Wheter or not to enable the detection of 3D movies playing
 	///
 	XBMCVideoChecker(const std::string & address, uint16_t port, bool grabVideo, bool grabPhoto, bool grabAudio, bool grabMenu, bool grabScreensaver, bool enable3DDetection);
@@ -96,6 +96,12 @@ private slots:
 	/// The JSON-RPC message to check the screensaver
 	const QString _checkScreensaverRequest;
 
+	/// The JSON-RPC message to check the active stereoscopicmode
+	const QString _getStereoscopicMode;
+
+	/// The JSON-RPC message to check the xbmc version
+	const QString _getXbmcVersion;
+
 	/// The QT TCP Socket with connection to XBMC
 	QTcpSocket _socket;
 
@@ -111,7 +117,7 @@ private slots:
 	/// Flag indicating whether or not to grab when XBMC is playing nothing (in menu)
 	const bool _grabMenu;
 
-	/// Flag inidcating whether or not to grab when the XBMC screensaver is activated
+	/// Flag indicating whether or not to grab when the XBMC screensaver is activated
 	const bool _grabScreensaver;
 
 	/// Flag indicating wheter or not to enable the detection of 3D movies playing
@@ -125,4 +131,7 @@ private slots:
 
 	/// Previous emitted video mode
 	VideoMode _previousVideoMode;
+
+	/// XBMC version number
+	int _xbmcVersion;
 };
diff --git a/libsrc/effectengine/Effect.cpp b/libsrc/effectengine/Effect.cpp
@@ -64,6 +64,9 @@ Effect::Effect(PyThreadState * mainThreadState, int priority, int timeout, const
 {
 	_colors.resize(_imageProcessor->getLedCount(), ColorRgb::BLACK);
 
+	// disable the black border detector for effects
+	_imageProcessor->enableBalckBorderDetector(false);
+
 	// connect the finished signal
 	connect(this, SIGNAL(finished()), this, SLOT(effectFinished()));
 }
diff --git a/libsrc/hyperion/ImageProcessor.cpp b/libsrc/hyperion/ImageProcessor.cpp
@@ -43,6 +43,11 @@ void ImageProcessor::setSize(const unsigned width, const unsigned height)
 	_imageToLeds = new ImageToLedsMap(width, height, 0, 0, _ledString.leds());
 }
 
+void ImageProcessor::enableBalckBorderDetector(bool enable)
+{
+	_enableBlackBorderRemoval = enable;
+}
+
 bool ImageProcessor::getScanParameters(size_t led, double &hscanBegin, double &hscanEnd, double &vscanBegin, double &vscanEnd) const
 {
 	if (led < _ledString.leds().size())
diff --git a/libsrc/hyperion/ImageToLedsMap.cpp b/libsrc/hyperion/ImageToLedsMap.cpp
@@ -35,15 +35,16 @@ ImageToLedsMap::ImageToLedsMap(
 		// skip leds without area
 		if ((led.maxX_frac-led.minX_frac) < 1e-6 || (led.maxY_frac-led.minY_frac) < 1e-6)
 		{
+			mColorsMap.emplace_back();
 			continue;
 		}
-		
+
 		// Compute the index boundaries for this led
 		unsigned minX_idx = xOffset + unsigned(std::round(actualWidth  * led.minX_frac));
 		unsigned maxX_idx = xOffset + unsigned(std::round(actualWidth  * led.maxX_frac));
 		unsigned minY_idx = yOffset + unsigned(std::round(actualHeight * led.minY_frac));
 		unsigned maxY_idx = yOffset + unsigned(std::round(actualHeight * led.maxY_frac));
-		
+
 		// make sure that the area is at least a single led large
 		minX_idx = std::min(minX_idx, xOffset + actualWidth - 1);
 		if (minX_idx == maxX_idx)
diff --git a/libsrc/leddevice/CMakeLists.txt b/libsrc/leddevice/CMakeLists.txt
@@ -13,6 +13,7 @@ include_directories(
 
 # Group the headers that go through the MOC compiler
 SET(Leddevice_QT_HEADERS
+		${CURRENT_SOURCE_DIR}/LedRs232Device.h
 		${CURRENT_SOURCE_DIR}/LedDeviceAdalight.h
 		${CURRENT_SOURCE_DIR}/LedDevicePhilipsHue.h
 )
@@ -21,8 +22,6 @@ SET(Leddevice_HEADERS
 		${CURRENT_HEADER_DIR}/LedDevice.h
 		${CURRENT_HEADER_DIR}/LedDeviceFactory.h
 
-		${CURRENT_SOURCE_DIR}/LedRs232Device.h
-
 		${CURRENT_SOURCE_DIR}/LedDeviceLightpack.h
 		${CURRENT_SOURCE_DIR}/LedDeviceMultiLightpack.h
 		${CURRENT_SOURCE_DIR}/LedDevicePaintpack.h
diff --git a/libsrc/leddevice/LedDeviceAdalight.cpp b/libsrc/leddevice/LedDeviceAdalight.cpp
@@ -11,8 +11,8 @@
 // hyperion local includes
 #include "LedDeviceAdalight.h"
 
-LedDeviceAdalight::LedDeviceAdalight(const std::string& outputDevice, const unsigned baudrate) :
-	LedRs232Device(outputDevice, baudrate),
+LedDeviceAdalight::LedDeviceAdalight(const std::string& outputDevice, const unsigned baudrate, int delayAfterConnect_ms) :
+	LedRs232Device(outputDevice, baudrate, delayAfterConnect_ms),
 	_ledBuffer(0),
 	_timer()
 {
diff --git a/libsrc/leddevice/LedDeviceAdalight.h b/libsrc/leddevice/LedDeviceAdalight.h
@@ -12,7 +12,7 @@
 ///
 /// Implementation of the LedDevice interface for writing to an Adalight led device.
 ///
-class LedDeviceAdalight : public QObject, public LedRs232Device
+class LedDeviceAdalight : public LedRs232Device
 {
 	Q_OBJECT
 
@@ -23,7 +23,7 @@ class LedDeviceAdalight : public QObject, public LedRs232Device
 	/// @param outputDevice The name of the output device (eg '/dev/ttyS0')
 	/// @param baudrate The used baudrate for writing to the output device
 	///
-	LedDeviceAdalight(const std::string& outputDevice, const unsigned baudrate);
+	LedDeviceAdalight(const std::string& outputDevice, const unsigned baudrate, int delayAfterConnect_ms);
 
 	///
 	/// Writes the led color values to the led-device
diff --git a/libsrc/leddevice/LedDeviceFactory.cpp b/libsrc/leddevice/LedDeviceFactory.cpp
@@ -43,8 +43,9 @@ LedDevice * LedDeviceFactory::construct(const Json::Value & deviceConfig)
 	{
 		const std::string output = deviceConfig["output"].asString();
 		const unsigned rate      = deviceConfig["rate"].asInt();
+		const int delay_ms       = deviceConfig["delayAfterConnect"].asInt();
 
-		LedDeviceAdalight* deviceAdalight = new LedDeviceAdalight(output, rate);
+		LedDeviceAdalight* deviceAdalight = new LedDeviceAdalight(output, rate, delay_ms);
 		deviceAdalight->open();
 
 		device = deviceAdalight;
diff --git a/libsrc/leddevice/LedDevicePhilipsHue.h b/libsrc/leddevice/LedDevicePhilipsHue.h
@@ -14,11 +14,13 @@
 
 /**
  * Implementation for the Philips Hue system.
- * 
+ *
  * To use set the device to "philipshue".
  * Uses the official Philips Hue API (http://developers.meethue.com).
  * Framegrabber must be limited to 10 Hz / numer of lights to avoid rate limitation by the hue bridge.
  * Create a new API user name "newdeveloper" on the bridge (http://developers.meethue.com/gettingstarted.html)
+ *
+ * @author ntim (github)
  */
 class LedDevicePhilipsHue: public QObject, public LedDevice {
 Q_OBJECT
diff --git a/libsrc/leddevice/LedRs232Device.cpp b/libsrc/leddevice/LedRs232Device.cpp
@@ -4,17 +4,21 @@
 #include <cstdio>
 #include <iostream>
 
+// Qt includes
+#include <QTimer>
+
 // Serial includes
 #include <serial/serial.h>
 
 // Local Hyperion includes
 #include "LedRs232Device.h"
 
-
-LedRs232Device::LedRs232Device(const std::string& outputDevice, const unsigned baudrate) :
-	mDeviceName(outputDevice),
-	mBaudRate_Hz(baudrate),
-	_rs232Port()
+LedRs232Device::LedRs232Device(const std::string& outputDevice, const unsigned baudrate, int delayAfterConnect_ms) :
+	_deviceName(outputDevice),
+	_baudRate_Hz(baudrate),
+	_delayAfterConnect_ms(delayAfterConnect_ms),
+	_rs232Port(),
+	_blockedForDelay(false)
 {
 	// empty
 }
@@ -31,10 +35,17 @@ int LedRs232Device::open()
 {
 	try
 	{
-		std::cout << "Opening UART: " << mDeviceName << std::endl;
-		_rs232Port.setPort(mDeviceName);
-		_rs232Port.setBaudrate(mBaudRate_Hz);
+		std::cout << "Opening UART: " << _deviceName << std::endl;
+		_rs232Port.setPort(_deviceName);
+		_rs232Port.setBaudrate(_baudRate_Hz);
 		_rs232Port.open();
+
+		if (_delayAfterConnect_ms > 0)
+		{
+			_blockedForDelay = true;
+			QTimer::singleShot(_delayAfterConnect_ms, this, SLOT(unblockAfterDelay()));
+			std::cout << "Device blocked for " << _delayAfterConnect_ms << " ms" << std::endl;
+		}
 	}
 	catch (const std::exception& e)
 	{
@@ -47,6 +58,11 @@ int LedRs232Device::open()
 
 int LedRs232Device::writeBytes(const unsigned size, const uint8_t * data)
 {
+	if (_blockedForDelay)
+	{
+		return 0;
+	}
+
 	if (!_rs232Port.isOpen())
 	{
 		return -1;
@@ -95,3 +111,9 @@ int LedRs232Device::writeBytes(const unsigned size, const uint8_t * data)
 
 	return 0;
 }
+
+void LedRs232Device::unblockAfterDelay()
+{
+	std::cout << "Device unblocked" << std::endl;
+	_blockedForDelay = false;
+}
diff --git a/libsrc/leddevice/LedRs232Device.h b/libsrc/leddevice/LedRs232Device.h
@@ -1,5 +1,7 @@
 #pragma once
 
+#include <QObject>
+
 // Serial includes
 #include <serial/serial.h>
 
@@ -9,16 +11,18 @@
 ///
 /// The LedRs232Device implements an abstract base-class for LedDevices using a RS232-device.
 ///
-class LedRs232Device : public LedDevice
+class LedRs232Device : public QObject, public LedDevice
 {
+	Q_OBJECT
+
 public:
 	///
 	/// Constructs the LedDevice attached to a RS232-device
 	///
 	/// @param[in] outputDevice The name of the output device (eg '/etc/ttyS0')
 	/// @param[in] baudrate The used baudrate for writing to the output device
 	///
-	LedRs232Device(const std::string& outputDevice, const unsigned baudrate);
+	LedRs232Device(const std::string& outputDevice, const unsigned baudrate, int delayAfterConnect_ms = 0);
 
 	///
 	/// Destructor of the LedDevice; closes the output device if it is open
@@ -43,12 +47,22 @@ class LedRs232Device : public LedDevice
 	 */
 	int writeBytes(const unsigned size, const uint8_t *data);
 
+private slots:
+	/// Unblock the device after a connection delay
+	void unblockAfterDelay();
+
 private:
 	/// The name of the output device
-	const std::string mDeviceName;
+	const std::string _deviceName;
+
 	/// The used baudrate of the output device
-	const int mBaudRate_Hz;
+	const int _baudRate_Hz;
+
+	/// Sleep after the connect before continuing
+	const int _delayAfterConnect_ms;
 
 	/// The RS232 serial-device
 	serial::Serial _rs232Port;
+
+	bool _blockedForDelay;
 };
diff --git a/libsrc/utils/CMakeLists.txt b/libsrc/utils/CMakeLists.txt
@@ -11,6 +11,7 @@ add_library(hyperion-utils
 		${CURRENT_HEADER_DIR}/ColorRgba.h
 		${CURRENT_SOURCE_DIR}/ColorRgba.cpp
 		${CURRENT_HEADER_DIR}/Image.h
+		${CURRENT_HEADER_DIR}/Sleep.h
 
 		${CURRENT_HEADER_DIR}/HsvTransform.h
 		${CURRENT_SOURCE_DIR}/HsvTransform.cpp
diff --git a/libsrc/xbmcvideochecker/XBMCVideoChecker.cpp b/libsrc/xbmcvideochecker/XBMCVideoChecker.cpp
diff --git a/src/hyperiond/hyperiond.cpp b/src/hyperiond/hyperiond.cpp

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,9 @@ Effect::Effect(PyThreadState * mainThreadState, int priority, int timeout, const`
`64`	`64`	`{`
`65`	`65`	`_colors.resize(_imageProcessor->getLedCount(), ColorRgb::BLACK);`
`66`	`66`
	`67`	`+ // disable the black border detector for effects`
	`68`	`+ _imageProcessor->enableBalckBorderDetector(false);`
	`69`	`+`
`67`	`70`	`// connect the finished signal`
`68`	`71`	`connect(this, SIGNAL(finished()), this, SLOT(effectFinished()));`
`69`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -43,6 +43,11 @@ void ImageProcessor::setSize(const unsigned width, const unsigned height)`
`43`	`43`	`_imageToLeds = new ImageToLedsMap(width, height, 0, 0, _ledString.leds());`
`44`	`44`	`}`
`45`	`45`
	`46`	`+void ImageProcessor::enableBalckBorderDetector(bool enable)`
	`47`	`+{`
	`48`	`+ _enableBlackBorderRemoval = enable;`
	`49`	`+}`
	`50`	`+`
`46`	`51`	`bool ImageProcessor::getScanParameters(size_t led, double &hscanBegin, double &hscanEnd, double &vscanBegin, double &vscanEnd) const`
`47`	`52`	`{`
`48`	`53`	`if (led < _ledString.leds().size())`