Blender Git Loki
Git Commits -> Revision 8051c26
Revision 8051c26 by Sergey Sharybin (cycles-x) March 11, 2021, 14:01 (GMT) |
Cycles: Rename Denoiser to DenoiserPipeline The goal is to have clarity on entity names. The proposed/desired convention: - Device: low-level "glue" logic, which takes care of running functions on a particular device. Algorithm implementation does not belong to device. - Denoiser: breaks down specific denoising algorithm into a function calls which happens on a device. It does not belong to the device because denoising can happen regardless of where noisy image came from: could be file, could be single render buffer, could be multiple render buffers rendered by multiple devices. - DenoiserPipeline: takes care of performing all steps needed to denoise a sequence of frames. In an ideal world DenoiserPipeline will implement a frame accessor of some sort, and use Denoiser to perform actual denoising on a configured device/denoising algorithm. This change only modifies the class name, making room for implementing an actual denoising algorithms as dedicated entities. Further cleanup in naming and structure is needed. It will be revisited once the cross-frame denoising will be worked on as an official core functionality. |
Commit Details:
Full Hash: 8051c2665326d2ee1ef09b09b4b5caf0f2b3885d
Parent Commit: e3f0fb3
Lines Changed: +17, -11