-
Notifications
You must be signed in to change notification settings - Fork 404
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Clang Format add "global" table to Lua state plus redo voice data for formula editor Default sandbox functions Add back "global" table clear in setSurgeFunctionEnvironment() Add stack test squash Fix segfault in testrunner clang format Move Nimbus from libsamplerate to our Lanczos downsampler (#7726) 1. Makes it ready to move to sst-effects 2. Makes it so we can almost remove libsamplerate (but some twist smidgeof work to do that) 3. Uses less CPU Addresses #7359 Remove libsamplerate dependency (#7728) After porting Nimbus ot lanczos in 4b3ffe6 we can now remove the libsamplerate dependency altogether as long as we fix up the twist fm (which we did with a simple application of another lanczos ds) and remove src code from the unit tests (where it was explicitly tested) Move Nimbus to sst-effects (#7729) * Move Nimbus to sst-effects Addresses #7359 With an almost-complete port (param name dynamism is in surge but not in the submodule yet) * f Update LFO and Step Seq presets to have the LFO EG enabled (#7732) Code Checks github action, as part one of move to actions Move to actions here will be a bit trickier, but we will get there. Step one is have any action at all, so add a code check action. Pull Request moves to Github Actions (#7733) The PullRequest support of our azure pipeline is now on GitHub actions, building in various configurations and running the tests. The azure pipeline still exists to trigger the release stage, which is what I'll port tomorrow, or maybe thursday, and then be done with the azure->actions project. - Adds voice_id, voice_max, voice_count, is_display parameters to formula modulator - Fakes voice_count of "1" and sets defaults for other params on display calls - Adds "shared" table to formula modulator - Sets entries of "shared" table to nil on new Lua function env - Makes formula debugger also use "shared" table - Adds lua_isfunction to formula debugger - Some cleanup
- Loading branch information
Showing
52 changed files
with
438 additions
and
954 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,106 @@ | ||
| name: Build Pull Request | ||
| on: | ||
| pull_request: | ||
|
|
||
| defaults: | ||
| run: | ||
| shell: bash | ||
|
|
||
| jobs: | ||
| build_plugin: | ||
| name: PR - ${{ matrix.name }} | ||
| runs-on: ${{ matrix.os }} | ||
| strategy: | ||
| matrix: | ||
| include: | ||
| - name: "windows msvc" | ||
| os: windows-latest | ||
| target: surge-xt_Standalone | ||
| cmakeConfig: -A x64 | ||
| cmakeOpt: DEBUG | ||
| runTests: false | ||
| - name: "windows msvc 32 bit" | ||
| os: windows-latest | ||
| target: surge-xt_Standalone | ||
| cmakeConfig: -A Win32 | ||
| cmakeOpt: DEBUG | ||
| runTests: false | ||
| - name: "windows clang" | ||
| os: windows-latest | ||
| target: surge-xt_Standalone | ||
| cmakeConfig: -DCMAKE_CXX_COMPILER=clang++ -DCMAKE_C_COMPILER=clang | ||
| cmakeOpt: DEBUG | ||
| runTests: false | ||
| - name: "ubuntu standalone" | ||
| os: ubuntu-latest | ||
| target: surge-xt_Standalone | ||
| cmakeConfig: | ||
| cmakeOpt: DEBUG | ||
| runTests: false | ||
| - name: "macos standalone" | ||
| os: macos-latest | ||
| target: surge-xt_Standalone | ||
| cmakeConfig: -DCMAKE_OSX_ARCHITECTURES="x86_64;arm64" | ||
| cmakeOpt: DEBUG | ||
| runTests: false | ||
|
|
||
| - name: "mac test runner" | ||
| os: macos-latest | ||
| target: surge-testrunner | ||
| cmakeConfig: | ||
| cmakeOpt: RELEASE | ||
| runTests: true | ||
| - name: "linux test runner" | ||
| os: ubuntu-latest | ||
| target: surge-testrunner | ||
| cmakeConfig: | ||
| cmakeOpt: RELEASE | ||
| runTests: true | ||
| - name: "windows test runner" | ||
| os: windows-latest | ||
| target: surge-testrunner | ||
| cmakeConfig: -A x64 | ||
| cmakeOpt: RELEASE | ||
| runTests: true | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| submodules: recursive | ||
|
|
||
| - name: Prepare for JUCE | ||
| uses: surge-synthesizer/sst-githubactions/prepare-for-juce@main | ||
| with: | ||
| os: ${{ runner.os }} | ||
|
|
||
| - name: Build pull request version | ||
| run: | | ||
| cmake -S . -B ./build -DCMAKE_BUILD_TYPE=${{ matrix.cmakeOpt }} ${{ matrix.cmakeConfig }} | ||
| cmake --build ./build --config ${{ matrix.cmakeOpt }} --target ${{ matrix.target }} --parallel 3 | ||
| - name: Run Tests | ||
| if: ${{ matrix.runTests }} | ||
| run: | | ||
| set -e | ||
| cd build | ||
| ctest -j 4 || ctest --rerun-failed --output-on-failure | ||
| build_plugin_lindoc: | ||
| name: PR - U18 Docker | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| submodules: recursive | ||
|
|
||
| - name: Build in Docker | ||
| uses: surge-synthesizer/sst-githubactions/cmake-in-docker@main | ||
| with: | ||
| username: ${{ github.actor }} | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| cmakeArgs: -DCMAKE_BUILD_TYPE=Debug -GNinja | ||
| target: surge-xt_Standalone | ||
|
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| name: Format Check | ||
| on: [ pull_request ] | ||
| jobs: | ||
| formatting-check: | ||
| name: Clang Format Check | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| path: [ 'src' ] | ||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v3 | ||
|
|
||
| - name: Run clang-format style check | ||
| uses: surge-synthesizer/sst-githubactions/clang-format-check@main | ||
| with: | ||
| path: ${{ matrix.path }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Submodule libsamplerate
deleted from
d0ea53
Submodule sst-basic-blocks
updated
4 files
| +87 −0 | include/sst/basic-blocks/dsp/RNG.h | |
| +4 −1 | include/sst/basic-blocks/modulators/FXModControl.h | |
| +26 −12 | include/sst/basic-blocks/modulators/SimpleLFO.h | |
| +7 −1 | tests/smoketest.cpp |
Submodule sst-effects
updated
13 files
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.