Add external DRED weights and harden borrowed wrappers

[Deniskore]

No description provided.

[Copilot]

Pull request overview

Adds runtime-loaded DRED weights and prevents borrowed wrappers from exposing movable owning handles.

Changes:

• Adds external-weight loading, aligned retention, build configuration, and end-to-end verification.
• Replaces borrowed-wrapper DerefMut access with explicit forwarding methods.
• Hardens packet validation and expands CTL tests.

Reviewed changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
.github/workflows/ci.yml	Adds external-weight CI coverage.
Cargo.lock	Refreshes dependency resolutions.
Cargo.toml	Defines the external-weights feature.
README.md	Documents DRED feature behavior.
build.rs	Configures and downloads external-weight assets.
scripts/verify_external_weights.py	Generates weights and runs integration testing.
src/decoder.rs	Adds retained decoder weights and hardened borrowing.
src/dred.rs	Copies and retains DRED decoder weights.
src/encoder.rs	Adds retained encoder weights and hardened borrowing.
src/lib.rs	Adds borrowed-method delegation macros.
src/multistream.rs	Adds LSB-depth CTLs and safe delegation.
src/packet.rs	Adds overflow and parser-result validation.
src/projection.rs	Hardens borrowed projection wrappers.
src/repacketizer.rs	Hardens borrowed repacketizer access.
tests/ctl_compatibility.rs	Expands CTL compatibility assertions.
tests/external_weights.rs	Tests external DRED weights end to end.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Pull request overview

Copilot reviewed 15 out of 16 changed files in this pull request and generated no new comments.

Suppressed comments (1)

src/encoder.rs:701

• Once external weights are active, this makes every setter routed through simple_ctl (including bitrate and packet-loss updates) parse and initialize the entire DNN blob again. Those CTLs may be adjusted on the real-time path, so they should remain constant-time; libopus keeps the model fields outside its reset ranges, so this reload is not needed for ordinary setters. Remove it here and, if a particular operation truly clears the model, restore only around that operation.

        self.reload_active_dnn_blob()?;