Skip to content

[Casual Forcing] Simplify the demo integration + remove runner - #462

Draft
ArielG-NV wants to merge 1 commit into
NVIDIA:mainfrom
ArielG-NV:api-refactor/simplify-casual-forcing
Draft

[Casual Forcing] Simplify the demo integration + remove runner#462
ArielG-NV wants to merge 1 commit into
NVIDIA:mainfrom
ArielG-NV:api-refactor/simplify-casual-forcing

Conversation

@ArielG-NV

Copy link
Copy Markdown
Collaborator

Build model then to render as a MP4 via custom code in 200LOC. Now you can integrate into MP4, WebRTC, and a Native-Slangpy-Window all in ~15LOC.

…n 200LOC. Now you can integrate into MP4, WebRTC, and a Native-Slangpy-Window all in ~15LOC.
@copy-pr-bot

copy-pr-bot Bot commented Aug 14, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@greptile-apps

greptile-apps Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR replaces Causal-Forcing’s model-specific runners with the shared T2V application and keeps its pipeline configurations available as programmatic presets.

  • Registers t2v-causal-forcing as an application entry point supporting local-window, MP4, and WebRTC outputs.
  • Removes the custom runner implementation, runner entry points, and now-unused package dependencies.
  • Updates smoke tests and documentation for the simplified integration.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete blocking or non-blocking defects identified in the changed integration path.

The shared application retains valid session construction, pipeline defaults, tensor layout, and entry-point discovery, while repository references and dependency metadata are consistent with removal of the custom runners.

Important Files Changed

Filename Overview
integrations/causal_forcing/causal_forcing/t2v/app.py Replaces the integration-specific application subclass with a direct shared T2VApplication factory using valid explicit defaults.
integrations/causal_forcing/causal_forcing/config.py Removes runner configurations while retaining three named pipeline presets and a consistent preset registry.
integrations/causal_forcing/causal_forcing/runner.py Deletes the custom T2V/I2V runner after application-based T2V integration and programmatic pipeline presets replace its supported roles.
integrations/causal_forcing/pyproject.toml Replaces runner entry points with one shared-application entry point and removes dependencies unused after runner deletion.
integrations/causal_forcing/tests/test_smoke.py Updates smoke coverage to validate pipeline naming, application defaults, and installed application discovery.
uv.lock Synchronizes Causal-Forcing package metadata with the removed direct dependencies without changing package versions.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  User[flashdreams-run t2v-causal-forcing] --> Registry[Application entry-point registry]
  Registry --> Factory[causal_forcing.t2v.app:create_app]
  Factory --> SharedApp[Shared T2VApplication]
  SharedApp --> Preset[Chunkwise Causal-Forcing pipeline preset]
  Preset --> Output{Output backend}
  Output --> Window[Native window]
  Output --> MP4[MP4 file]
  Output --> WebRTC[WebRTC session]
Loading

Reviews (1): Last reviewed commit: "simplify casual forcing - build model th..." | Re-trigger Greptile

@ArielG-NV
ArielG-NV marked this pull request as draft August 17, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant