Compositor: Add OpenImageDenoising prefiltering option to Denoising node. #90919

New Issue

Closed

opened 2021-08-25 16:44:04 +02:00 by Jeroen Bakker · 5 comments

Jeroen Bakker commented

2021-08-25 16:44:04 +02:00

Member

CyclesX will have options to use OIDN with a prefiltering option.
This prefiltering option we want to add to the compositor as well.

Challenge of this ticket is that prefiltering takes time and commonly the same prefiltering result can be used by multiple nodes.

Proposal

Add new DenoisePrefilterOperation that prefilters the input images.
This operation is added just before the denoise operation when the prefiltering is enabled.
Add a compiler post step that finds prefilter operations that share the same settings/inputs and merge them.

D12043: Cycles X: Support OIDN with guiding pass prefiltering
D12140: Cycles X: More flexible OIDN prefiltering settings

CyclesX will have options to use OIDN with a prefiltering option. This prefiltering option we want to add to the compositor as well. Challenge of this ticket is that prefiltering takes time and commonly the same prefiltering result can be used by multiple nodes. ## Proposal * Add new DenoisePrefilterOperation that prefilters the input images. * This operation is added just before the denoise operation when the prefiltering is enabled. * Add a compiler post step that finds prefilter operations that share the same settings/inputs and merge them. ## Related patches [D12043: Cycles X: Support OIDN with guiding pass prefiltering](https://archive.blender.org/developer/D12043) [D12140: Cycles X: More flexible OIDN prefiltering settings](https://archive.blender.org/developer/D12140)

Jeroen Bakker commented

2021-08-25 16:44:04 +02:00

Author

Member

Changed status from 'Needs Triage' to: 'Confirmed'

Changed status from 'Needs Triage' to: 'Confirmed'

Jeroen Bakker commented

2021-08-25 16:44:04 +02:00

Author

Member

Added subscriber: @Jeroen-Bakker

Added subscriber: @Jeroen-Bakker

Manuel Castilla was assigned by Jeroen Bakker

2021-08-25 17:02:30 +02:00

Evan Wilson commented

2021-08-25 17:19:02 +02:00

Member

Added subscriber: @EAW

Added subscriber: @EAW

blender-admin commented

2021-09-19 20:12:53 +02:00

This issue was referenced by 276eebb274

This issue was referenced by 276eebb274744d819dcdab8a95770dd7382c0664

Aaron Carlisle commented

2021-09-22 23:08:09 +02:00

Member

Changed status from 'Confirmed' to: 'Resolved'

Changed status from 'Confirmed' to: 'Resolved'

Aaron Carlisle closed this issue

2021-09-22 23:08:09 +02:00

Thomas Dinges added this to the 3.0 milestone 2023-02-08 15:59:05 +01:00

Sign in to join this conversation.

No Label

Animation & Rigging

Asset Browser Project Overview

Automated Testing

Blender Asset Bundle

Dependency Graph

Development Management

EEVEE & Viewport

Images & Movies

Motion Tracking

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Render Pipeline

Sculpt, Paint & Texture

Video Sequencer

Virtual Reality

Blender 2.8 Project

Milestone 1: Basic, Local Asset Browser

Good First Issue

Animation & Rigging

Development Management

EEVEE & Viewport

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Sculpt, Paint & Texture

Needs Info from Developers

Needs Information from User

No Milestone

No project

No Assignees

4 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#90919

No description provided.