-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for null / phase cancellation testing #10
Comments
This tool comes close to accomplishing all of these: https://github.com/benthevining/plugin-cancellation-testing It's a wrapper CMake script on top of Plugalyzer, with another tool I wrote to do the comparison. I still need to add actually generating a diff file, and integration with your sparklines module is a good idea too! |
I could eliminate the need for my secondary tool if plugalyzer itself could do the comparison and thresholding to pass/fail the test. Even if this feature is added to plugalyzer, I still think having some wrapper cmake code is valuable, because you can drive regenerating the reference files using the test definitions. |
I don't understand what you mean by this. I am open to suggestions from both of you on how to integrate this into Plugalyzer, but I'm wondering whether it should even live in Plugalyzer itself. Are there any other types of validation that we might want to perform on the output files in the future? This could indicate whether to add this directly or implement a broader validation API. |
I'm open to integrating my cmake code into plugalyzer, if you're interested in adding the diff testing features directly to the plugalyzer CLI app. (But my code continuing to live in a separate repo is fine too.) Basically it would be a CMake module that people could use somewhat like: find_package (Plugalyzer REQUIRED)
Plugalyzer_add_cancellation_test (
myPlugin_VST3
INPUT_AUDIO input.wav
REFERENCE_AUDIO output.wav
STATE_FILE state.json
EXACT
REGEN_TARGET RegenerateReferenceAudio
) This CMake function is useful because:
I think it's quite valuable to be able to use your test definitions to also drive regenerating the reference files, and that's one of the main features this code provides. |
Possibly. For a while I've been brainstorming some kind of tool to detect audio discontinuities, but I don't have any solid design/implementation of that yet... |
This is most likely 100% out of scope for the time being and might belong in a tool that uses plugalyzer.
I'd like to store reference audio files in my repository. In CI, I will compare plugalyzer's current output to those reference files for regression testing.
So the feature would be: After creation of an output file, create a "difference" file (invert the phase of the current output, add it to the reference).
This would require
The text was updated successfully, but these errors were encountered: