Pull Request: Fix protobuf dependency conflict: Replace descript-audiotools with latentforge/audiotools#10
Open
wzgrx wants to merge 1 commit intoDragonDiffusionbyBoyo:mainfrom
Open
Pull Request: Fix protobuf dependency conflict: Replace descript-audiotools with latentforge/audiotools#10wzgrx wants to merge 1 commit intoDragonDiffusionbyBoyo:mainfrom
wzgrx wants to merge 1 commit intoDragonDiffusionbyBoyo:mainfrom
Conversation
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem Description
The official descript-audiotools (0.7.2) enforces protobuf=4.25.1, making it impossible for the environment to support both audio processing and model inference simultaneously.
Solution
Remove official version: Delete descript-audiotools (known conflict source)
Introduce patched version: Add latentforge/audiotools (GitHub Fork version), which:
Retains all core functionality of original (AudioSignal, processing pipelines)
Updates dependency requirements to allow protobuf=3.20.0
protobuf>=4.25.1
Cleaned up duplicate dependencies:
Removed duplicate soundfile>=0.12.0 and hydra-core
Kept descript-audio-codec (no protobuf conflict)
Verification Results
Installation verification:
pip install -r requirements.txt
Installation completed without errors, no protobuf version conflicts
Version check:
pip show protobuf descript-audiotools
protobuf version: 4.25.1+ (meets onnx requirements)
descript-audiotools version: 0.7.4 (installed from GitHub)
Functionality verification:
Audio processing (pyloudnorm, librosa) working normally
ONNX model loading no ImportError
ComfyUI startup no dependency conflicts
Potential Impact
Positive impact:
Completely resolves the protobuf version deadlock
Allows simultaneous use of modern AI libraries and audio processing functionality
No need to sacrifice any features of ONNX or AudioTools
Considerations:
In rare cases, some plugins might depend on private APIs of the original 0.7.2 version
If encountering AttributeError, check plugin compatibility
Test Environment
Operating System: Windows 10
Python Version: 3.10
Key Library Versions:
torch: 2.11.0
onnxruntime: 1.18.0
protobuf: 4.25.1
This modification has been verified in a real-world environment and is strongly recommended for merging to resolve this long-standing dependency conflict.