Skip to content

Feature: Video Producer #86

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Feature: Video Producer #86

wants to merge 15 commits into from

Conversation

@fabioseel
Copy link
Contributor

@fabioseel fabioseel commented Oct 30, 2025 •
edited
Loading

  • extends the video producer script to allow for synchronous generation of RAW, DECODED, AUGMENTED and VALUE_MASK videos.
  • the value mask uses a more general analysis available in the analysis module, which will be helpful for (Attribution-)Masked Reconstruction Loss #74
  • also removes a script that for whatever reason made it to be commited to the top level directory (shame on me)

@fabioseel fabioseel changed the title Feat video producer Feature: Video Producer Oct 31, 2025
@fabioseel fabioseel requested a review from alex404 October 31, 2025 10:40
fabioseel
fabioseel commented Nov 4, 2025
View reviewed changes
fabioseel
fabioseel commented Nov 4, 2025
View reviewed changes
@fabioseel
Copy link
Contributor Author

fabioseel commented Nov 5, 2025

contains code relevant for #74

@fabioseel fabioseel force-pushed the feat-video-producer branch from 4774d45 to 4c6ddbe Compare November 5, 2025 20:20
@fabioseel
refactor: adjust save paths, add argparse (partially unused)
14d25d7
@fabioseel
refactor: rename (experiment_)cfg variable
e5ab978
@fabioseel
feat: enable selection of actor frame rate instead of full
bd257b9
@fabioseel
feat: enable different vid types (currently raw and augmented)
fb52d99
@fabioseel
feat: add decoder video output
3dbf740
@fabioseel
fix: correctly access responses out
518959f
@fabioseel
feat: add value mask video, fix: dont rely on naming conventions for …
fdcf97f 
…decoder out
@fabioseel
refactoring: factor attribution code into analysis+add functionality
dd060e2
@fabioseel
refactor: move rescale function to util
64319bc
@fabioseel
fix: if agent dies and VideoType.RAW is not used
1bda82e
@fabioseel
chore: remove duplicated script
140c08a
@fabioseel
fix: support for npz files in produce_rf_plot
8a6dc38
@fabioseel
chore: remove prints
81d915c
@fabioseel
fix: accidentally used vision input instead of decoder output
605ab8a
@fabioseel fabioseel force-pushed the feat-video-producer branch from 4c6ddbe to 605ab8a Compare November 5, 2025 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alex404 alex404 Awaiting requested review from alex404

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fabioseel