Auto build update (#366)

* - cherry picked logic server fix into master * added status badges [no-ci] * - cleaned up unit tests && added run_unit_test commandline tool * - removed annoying SNEX test files - added unit test compilation to CI export - added build_ci.bat script * - fixed some compile-time routing matrix classes * - update build script * - update macOS build script Co-authored-by: Christoph Hart <[email protected]>
christophhart · Oct 25, 2022 · 03a8568 · 03a8568
1 parent cf4d6c1
commit 03a8568
Show file tree

Hide file tree

Showing 28 changed files with 270 additions and 689 deletions.
diff --git a/README.md b/README.md
@@ -2,6 +2,11 @@
 
 ![](http://hise.audio/images/github.png)
 
+**Build Status macOS:**
+[![build_hise_macos](https://github.com/christophhart/HISE/actions/workflows/build_hise_macos.yml/badge.svg)](https://github.com/christophhart/HISE/actions/workflows/build_hise_macos.yml)
+
+**Build Status Windows:**
+[![build_hise](https://github.com/christophhart/HISE/actions/workflows/build_hise.yml/badge.svg)](https://github.com/christophhart/HISE/actions/workflows/build_hise.yml)
 
 **The open source framework for sample based instruments.**
 

diff --git a/hi_backend/snex_workbench/WorkbenchProcessor.cpp b/hi_backend/snex_workbench/WorkbenchProcessor.cpp
@@ -843,13 +843,15 @@ void DspNetworkCompileExporter::writeDebugFileAndShowSolution()
 
 	auto isUsingVs2017 = HelperClasses::isUsingVisualStudio2017(settings);
 
-	auto vsString = isUsingVs2017 ? "VisualStudio2017" : "VisualStudio2022";
-	auto vsVersion = isUsingVs2017 ? "15.0" : "17.0";
+
 
 	auto currentExecutable = File::getSpecialLocation(File::currentExecutableFile);
 
 
 #if JUCE_WINDOWS
+    auto vsString = isUsingVs2017 ? "VisualStudio2017" : "VisualStudio2022";
+    auto vsVersion = isUsingVs2017 ? "15.0" : "17.0";
+
     debugExecutable = debugExecutable.getChildFile(vsString).getChildFile("x64/Debug/App/HISE Debug.exe");
 
 	// If this hits, then you have a mismatch between VS2022 and VS2017...

diff --git a/hi_core/hi_core/StandaloneProcessor.cpp b/hi_core/hi_core/StandaloneProcessor.cpp
@@ -124,17 +124,7 @@ StandaloneProcessor::StandaloneProcessor()
 
 
 
-#if HI_RUN_UNIT_TESTS
 
-	UnitTestRunner runner;
-
-	runner.setAssertOnFailure(false);
-
-	runner.runAllTests();
-
-
-
-#endif
 
 
 	LOG_START("Create Main Processor");

diff --git a/hi_dsp_library/dsp_nodes/RoutingNodes.h b/hi_dsp_library/dsp_nodes/RoutingNodes.h
@@ -615,11 +615,9 @@ struct matrix_helpers
 			break;
 		case SpecialType::LeftToRight:
 			FloatVectorOperations::copy(ptrs[1], ptrs[0], d.getNumSamples());
-			FloatVectorOperations::clear(ptrs[0], d.getNumSamples());
 			break;
 		case SpecialType::RightToLeft:
 			FloatVectorOperations::copy(ptrs[0], ptrs[1], d.getNumSamples());
-			FloatVectorOperations::clear(ptrs[1], d.getNumSamples());
 			break;
 		case SpecialType::SwapChannels:
 		{
@@ -641,9 +639,9 @@ struct matrix_helpers
 		constexpr int r = MatrixType::getChannel(1);
 
 		if constexpr (l == 0 && r == 0)
-			return SpecialType::RightToLeft;
-		else if constexpr (l == 1 && r == 1)
 			return SpecialType::LeftToRight;
+		else if constexpr (l == 1 && r == 1)
+			return SpecialType::RightToLeft;
 		else if constexpr (l == 0 && r == -1)
 			return SpecialType::LeftOnly;
 		else if constexpr (l == -1 && r == 1)

diff --git a/hi_modules/synthesisers/synths/NoiseSynth.cpp b/hi_modules/synthesisers/synths/NoiseSynth.cpp
@@ -65,157 +65,6 @@ void NoiseVoice::calculateBlock(int startSample, int numSamples)
 	const int startIndex = startSample;
 	const int samplesToCopy = numSamples;
 
-#if HI_RUN_UNIT_TESTS
-
-	auto signalType = static_cast<NoiseSynth*>(getOwnerSynth())->getTestSignal();
-
-	switch (signalType)
-	{
-	case hise::NoiseSynth::Normal:
-	{
-		while (--numSamples >= 0)
-		{
-			const float currentSample = getNextValue();
-
-			// Stereo mode assumed
-			voiceBuffer.setSample(0, startSample, currentSample);
-
-			voiceUptime += uptimeDelta;
-
-			++startSample;
-		}
-		break;
-	}
-	case hise::NoiseSynth::DC:
-	{
-		FloatVectorOperations::fill(voiceBuffer.getWritePointer(0, startSample), 1.0f, samplesToCopy);
-		break;
-	}
-
-	case hise::NoiseSynth::Ramp:
-	{
-		while (--numSamples >= 0)
-		{
-			const float delta1 = 1.0f / (float)INT16_MAX;
-			const float uptimeToUse = (float)((int)voiceUptime % INT16_MAX);
-			const float value = uptimeToUse * delta1;
-
-			voiceBuffer.setSample(0, startSample, value);
-
-			voiceUptime += uptimeDelta;
-			++startSample;
-		}
-
-		break;
-	}
-	case hise::NoiseSynth::DiracTrain:
-	{
-		if (auto voicePitchValues = getOwnerSynth()->getPitchValuesForVoice())
-		{
-			voicePitchValues += startSample;
-
-			while (--numSamples >= 0)
-			{
-				if (voiceUptime == 0.0 && lastUptime == -1.0)
-					voiceBuffer.setSample(0, startSample, -1.0f);
-				else if (voiceUptime >= 256.0)
-				{
-					voiceUptime -= 256.0;
-					voiceBuffer.setSample(0, startSample, 1.0f);
-				}
-				else
-					voiceBuffer.setSample(0, startSample, 0.0f);
-
-				voiceUptime += uptimeDelta * *voicePitchValues++;
-				lastUptime = voiceUptime;
-				++startSample;
-			}
-		}
-		else
-		{
-			while (--numSamples >= 0)
-			{
-				if (voiceUptime == 0.0)
-					voiceBuffer.setSample(0, startSample, -1.0f);
-				else if (voiceUptime >= 256.0)
-				{
-					voiceUptime -= 256.0;
-					voiceBuffer.setSample(0, startSample, 1.0f);
-				}
-				else
-					voiceBuffer.setSample(0, startSample, 0.0f);
-
-				voiceUptime += uptimeDelta;
-				lastUptime = voiceUptime;
-				++startSample;
-			}
-		}
-
-
-
-		break;
-	}
-	case hise::NoiseSynth::Square:
-	{
-		if (auto voicePitchValues = getOwnerSynth()->getPitchValuesForVoice())
-		{
-			voicePitchValues += startSample;
-
-
-			while (--numSamples >= 0)
-			{
-				voiceUptime += uptimeDelta * *voicePitchValues++;
-
-				auto prevUptime = (int)(voiceUptime / 128.0);
-
-				if (prevUptime % 2 == 0)
-				{
-					voiceUptime -= 256.0;
-					voiceBuffer.setSample(0, startSample, 1.0f);
-				}
-				else
-				{
-					voiceBuffer.setSample(0, startSample, -1.0f);
-				}
-
-				++startSample;
-			}
-		}
-		else
-		{
-			while (--numSamples >= 0)
-			{
-				voiceUptime += uptimeDelta;
-
-				auto prevUptime = (int)(voiceUptime / 128.0);
-
-				if (prevUptime % 2 == 0)
-				{
-					voiceUptime -= 256.0;
-					voiceBuffer.setSample(0, startSample, 1.0f);
-				}
-				else
-				{
-					voiceBuffer.setSample(0, startSample, -1.0f);
-				}
-
-				++startSample;
-			}
-
-		}
-
-
-		break;
-	}
-
-	case hise::NoiseSynth::numTestSignals:
-		break;
-	default:
-		break;
-}
-
-#else
-
 	while (--numSamples >= 0)
 	{
 		const float currentSample = getNextValue();
@@ -228,9 +77,6 @@ void NoiseVoice::calculateBlock(int startSample, int numSamples)
 		++startSample;
 	}
 
-
-#endif
-
 	if (auto modValues = getOwnerSynth()->getVoiceGainValues())
 	{
 		FloatVectorOperations::multiply(voiceBuffer.getWritePointer(0, startIndex), modValues + startIndex, samplesToCopy);

diff --git a/hi_scripting/scripting/api/DspUnitTests.cpp b/hi_scripting/scripting/api/DspUnitTests.cpp
@@ -35,7 +35,6 @@
 #if HI_RUN_UNIT_TESTS
 
 #include  "JuceHeader.h"
-#include <ipp.h>
 
 using namespace hise;
 
@@ -206,8 +205,8 @@ class DspUnitTests : public UnitTest
 
 		Array<var> channels;
 
-		channels.add(var(lData));
-		channels.add(var(rData));
+		channels.add(var(lData.get()));
+		channels.add(var(rData.get()));
 
 		try
 		{
@@ -390,7 +389,7 @@ class ModulationTests : public UnitTest
 
 		// Setup
 
-		auto effect = Helpers::addVoiceEffectToOptionalGroup<StereoEffect>(bp);
+		Helpers::addVoiceEffectToOptionalGroup<StereoEffect>(bp);
 
 		auto lfo = Helpers::addTimeModulator<StereoEffect, LfoModulator>(bp, 0);
 
@@ -900,8 +899,8 @@ class ModulationTests : public UnitTest
 		squareLFO->setAttribute(LfoModulator::Parameters::FadeIn, 0.0f, dontSendNotification);
 		seqLFO->setAttribute(LfoModulator::Parameters::FadeIn, 0.0f, dontSendNotification);
 
-		seqLFO->getSliderPackData(0)->setNumSliders(2);
-		seqLFO->getSliderPackData(0)->setValue(1, 0.0f, dontSendNotification);
+		seqLFO->getSliderPack(0)->setNumSliders(2);
+		seqLFO->getSliderPack(0)->setValue(1, 0.0f, dontSendNotification);
 
 		// Process
 		squareLFO->setBypassed(true);

diff --git a/hi_scripting/scripting/api/GlobalServer.h b/hi_scripting/scripting/api/GlobalServer.h
@@ -215,7 +215,7 @@ struct GlobalServer: public ControlledObject
     }
 
 private:
-	
+
 #if USE_BACKEND
 	bool initialised = true;
 #else
-Original file line number
+Diff line change
@@ Expand Up / @@ -215,7 +215,7 @@ struct GlobalServer: public ControlledObject @@
         }
     private:
     #if USE_BACKEND
     	bool initialised = true;
     #else
@@ Expand Down @@