Blender Git Statistics -> Developers -> pmoursnv

Patrick Mours (pmoursnv)

Total Commits : 59
Master Commits : 57
Branch Commits : 2
First Commit : August 26, 2019
Latest Commit : March 26, 2020

Commits by Month

DateNumber of Commits
March, 20202
February, 202015
January, 20209
December, 20191
November, 20198
October, 20195
September, 20195
August, 201914

Commit Distribution

PathNumber of Commits
master57
vr_scene_inspection2

Favourite Files

FilenameTotal Edits
device_optix.cpp22
session.cpp7
kernel_path.h5
device_cuda_impl.cpp5
svm_bevel.h5
device_cuda.cpp5
device_multi.cpp5
kernel_types.h4
device_memory.h4
properties.py4

File Changes

ActionTotalPer Commit
Added100.2
Modified2454.2
Deleted20.0

Code Changes

ActionTotalPer Commit
Lines Added3 30266.0
Lines Removed2 01740.3

Latest commits Feed

Revision bd2b1a6 by Patrick Mours (master)
March 26, 2020, 12:00 (GMT)
Fix T74939: Random Walk subsurface appearance in OptiX does not match other engines

Random Walk subsurface scattering did look different with OptiX because transmittance is
calculated based on the hit distance, but the OptiX implementation of `scene_intersect_local`
would return the distance in world space, while the Cycles BVH version returns it in object
space. This fixes the problem by simply skipping the object->world transforms in all the
places using the result of `scene_intersect_local` with OptiX.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D7232
Revision 88db9a1 by Patrick Mours (master)
March 5, 2020, 15:28 (GMT)
Fix T74393: Cycles crashes when both OSL and Optix Denoising are enabled

Enabling viewport denoising causes Cycles to use a multi-device, which always returned NULL when
asked for OSL memory and would subsequently crash. This fixes that by returning the correct OSL
memory pointer from the CPU device in the special viewport denoising multi-device.
Revision af54bbd by Patrick Mours (master)
February 28, 2020, 15:12 (GMT)
Cycles: Rework tile scheduling for denoising

This fixes denoising being delayed until after all rendering has finished. Instead, tile-based
denoising is now part of the "RENDER" task again, so that it is all in one task and does not
cause issues with dedicated task pools where tasks are serialized.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D6940
Revision 0cea935 by Patrick Mours (master)
February 28, 2020, 14:58 (GMT)
Fix CUDA out of memory error with OptiX viewport denoising on small GPUs

This makes the memory allocation for the denoiser state use the memory allocator in Cycles, which
will evict textures to host memory when there is not enough space on the device. This means the
allocation for the denoiser state won't just fail if there is no more space and instead more space is
made for it to work. Also simplifies code somewhat.
Revision a930431 by Patrick Mours (master)
February 25, 2020, 16:13 (GMT)
Cleanup: Remove superfluous "cuda_device_ptr" function
Revision 0c09700 by Patrick Mours (master)
February 25, 2020, 14:27 (GMT)
Cycles: Add option to change which sample to start viewport denoising at

This patch adds a new user-configurable option to change at which sample viewport
denoising should kick in. Setting it to zero retains previous behavior (start immediately), while
other values will defer denoising until the particular sample has been reached. Default is now
at one, to avoid the weirdness that is AI denoising at small resolutions.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D6906
Revision f7b6b7d by Patrick Mours (master)
February 24, 2020, 16:53 (GMT)
Fix unnecessary Cycles OptiX kernel loading

With the OptiX viewport denoiser active, an OptiX device was added to the device list even when
rendering and denoising on different devices (e.g. CPU or CUDA) in background rendering. This
fixes it, so the OptiX device is only added when actually needed, as in only when OptiX denoising is
actually active in background rendering.
Revision 129fed6 by Patrick Mours / Julian Eisel (vr_scene_inspection)
February 20, 2020, 12:00 (GMT)
Cycles: Add support for adaptive kernel compilation to OptiX device

This modifies the common CUDA implementation for adaptive kernel compilation slightly to support both CUBIN and PTX output (the latter which is then used in the OptiX device). It also fixes adaptive kernel compilation on Windows.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D6851
Revision c093ef6 by Patrick Mours / Julian Eisel (vr_scene_inspection)
February 20, 2020, 12:00 (GMT)
Fix artifacts with Cycles viewport denoising when rendering with multiple CUDA devices

Rendering with multiple CUDA devices but denoising with OptiX caused parts of the image to go
missing at the start while the resolution was scaled. This is because the copy operation in
`MultiDevice::map_neighbor_tiles` which slices the copy across all devices would slice based on the
full resolution and not the scaled one and therefore copy incorrect data between devices.
Since this is not the recommended way of using viewport denoising anyway, simply avoid those
incorrect copies for now by disabling denoising while the resolution is scaled. Doing both rendering
and denoising with OptiX is not affected by this, since it avoids those copies altogether anyway.
Revision ab3a6e0 by Patrick Mours (master)
February 17, 2020, 15:15 (GMT)
Fix artifacts with Cycles viewport denoising when rendering with multiple CUDA devices

Rendering with multiple CUDA devices but denoising with OptiX caused parts of the image to go
missing at the start while the resolution was scaled. This is because the copy operation in
`MultiDevice::map_neighbor_tiles` which slices the copy across all devices would slice based on the
full resolution and not the scaled one and therefore copy incorrect data between devices.
Since this is not the recommended way of using viewport denoising anyway, simply avoid those
incorrect copies for now by disabling denoising while the resolution is scaled. Doing both rendering
and denoising with OptiX is not affected by this, since it avoids those copies altogether anyway.

MiikaHweb - Blender Git Statistics v1.06
By: Miika HämäläinenLast update: Nov-07-2014 14:18 MiikaHweb | 2003-2020