Skip to content
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

Add filter and transcode pipelines #1458

Merged
merged 3 commits into from
Dec 2, 2024
Merged

Conversation

jjnesbitt
Copy link
Member

@jjnesbitt jjnesbitt commented Nov 18, 2024

Exposes filter and transcode pipelines to the client. Results from these jobs are placed in the job work directory.

@jjnesbitt jjnesbitt marked this pull request as ready for review November 22, 2024 16:48
@jjnesbitt jjnesbitt requested a review from BryonLewis November 22, 2024 16:48
Comment on lines 129 to 134
// TODO: Investigate if any other types can use this.
if (runPipelineArgs.pipeline.type === 'filter') {
command.push(`-s kwa_writer:output_directory="${npath.join(jobWorkDir, 'output')}"`);
command.push('-s image_writer:file_name_template="frame%06d.jpg"');
command.push('-s image_writer:type="vxl"');
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is to accommodate some pipeline args that I noticed were missing. This afaik is not a complete list, but does allow for some pipelines like filter_to_kwa to be run.

@jjnesbitt jjnesbitt force-pushed the image-formatting-pipelines branch from c3d1821 to 4abae43 Compare November 25, 2024 18:37
@jjnesbitt jjnesbitt merged commit a1f79a4 into main Dec 2, 2024
4 checks passed
@jjnesbitt jjnesbitt deleted the image-formatting-pipelines branch December 2, 2024 15:17
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.

2 participants