Page MenuHome

Patrick Mours (pmoursnv)
User

Projects (12)

User Details

User Since
Feb 21 2019, 1:26 PM (169 w, 1 d)

Recent Activity

Yesterday

Patrick Mours (pmoursnv) updated the diff for D10023: Cycles: Add half precision float support to NanoVDB.

Sounds good, changed default volume precision to half and added the missing typenames.
Has the added benefit that there is now a nice correlation in the enum values between VDB_PRECISION_HALF/FULL/MINI_FLOAT from the fluid data block and VOLUME_PRECISION_HALF/FULL/VARIABLE.

Fri, May 20, 6:31 PM
Patrick Mours (pmoursnv) added a comment to D10023: Cycles: Add half precision float support to NanoVDB.

Since VolumeRender.precision already existed, existing files will have that set to zero (as it was never set to something else). I therefore defined VOLUME_PRECISION_VARIABLE as zero, so that the default for all files is to use variable bit quantization now. This could be changed to VOLUME_PRECISION_FULL being zero instead, so that existing files will use full precision like before and only new files will use variable bit quantization (because _DNA_DEFAULT_VolumeRender sets it to VOLUME_PRECISION_VARIABLE now).

Fri, May 20, 4:03 PM
Patrick Mours (pmoursnv) updated the diff for D10023: Cycles: Add half precision float support to NanoVDB.

Fixed build failure with Clang/GCC

Fri, May 20, 3:57 PM
Patrick Mours (pmoursnv) updated the diff for D10023: Cycles: Add half precision float support to NanoVDB.

Moved precision property into volume render data block.

Fri, May 20, 3:55 PM

Thu, May 19

Patrick Mours (pmoursnv) added a comment to D10023: Cycles: Add half precision float support to NanoVDB.

Can we make half precision a setting on the Volume datablock? Having to switch it for the entire scene for just one volume requiring additional precision is not ideal.

I wasn't entirely sure how, since there are no other Cycles settings in Volume and VolumeRender data blocks currently, other volume related settings are either in the material (e.g. material.cycles.volume_step_rate) or global render settings (e.g. scene.cycles.volume_step_rate). So not sure if it makes more sense to add to Volume/VolumeRender data block add add a new UI panel, or add it to the material and leverage the existing panel.

Thu, May 19, 3:47 PM

Tue, May 17

Patrick Mours (pmoursnv) updated the summary of D10023: Cycles: Add half precision float support to NanoVDB.
Tue, May 17, 4:54 PM
Patrick Mours (pmoursnv) updated the diff for D10023: Cycles: Add half precision float support to NanoVDB.

Reworked to use the new quantization data types in NanoVDB (nanovdb::Fp16, nanovdb::FpN), now that NanoVDB was updated.

Tue, May 17, 4:50 PM

Sat, May 7

Alaska (Alaska) awarded rB6fa5d520b861: Cycles: Add support for parallel compilation of OptiX module a Like token.
Sat, May 7, 2:35 AM

Fri, May 6

Patrick Mours (pmoursnv) closed T94354: Strange CyclesX Optix Motion Blur with rotating object as Resolved.

This was happening because of different intersection precision and self intersection and was fixed by rBae440703411486c9219fa0ff54e471eea64afb58.

Fri, May 6, 10:29 PM · Render & Cycles, BF Blender

Thu, May 5

Metin Seven (MetinSeven) awarded rB6fa5d520b861: Cycles: Add support for parallel compilation of OptiX module a Like token.
Thu, May 5, 8:53 PM
Patrick Mours (pmoursnv) closed D14845: Cycles: Add support for parallel compilation of OptiX module.
Thu, May 5, 2:36 PM
Patrick Mours (pmoursnv) committed rB6fa5d520b861: Cycles: Add support for parallel compilation of OptiX module (authored by Patrick Mours (pmoursnv)).
Cycles: Add support for parallel compilation of OptiX module
Thu, May 5, 2:36 PM
Patrick Mours (pmoursnv) added inline comments to D14845: Cycles: Add support for parallel compilation of OptiX module.
Thu, May 5, 11:11 AM
Patrick Mours (pmoursnv) updated the diff for D14845: Cycles: Add support for parallel compilation of OptiX module.

Changed error reporting to use last error from optixTaskExecute.

Thu, May 5, 11:09 AM

Wed, May 4

Patrick Mours (pmoursnv) requested review of D14845: Cycles: Add support for parallel compilation of OptiX module.
Wed, May 4, 4:49 PM

Apr 20 2022

Patrick Mours (pmoursnv) closed D14703: Cycles: Fix building USD support in Cycles standalone on Windows.
Apr 20 2022, 3:08 PM
Patrick Mours (pmoursnv) added inline comments to D14703: Cycles: Fix building USD support in Cycles standalone on Windows.
Apr 20 2022, 1:44 PM
Patrick Mours (pmoursnv) requested review of D14703: Cycles: Fix building USD support in Cycles standalone on Windows.
Apr 20 2022, 1:41 PM
Patrick Mours (pmoursnv) closed T73657: Benchmark GUI launcher does not run on Quadro cards as Resolved.

This turned out to be a driver issue that has been fixed in 512.15+.

Apr 20 2022, 11:13 AM · Blender Benchmark, Blender Open Data

Apr 19 2022

Patrick Mours (pmoursnv) awarded rC1a5f4932b481: Add USD as a file format for Cycles standalone rendering a Love token.
Apr 19 2022, 2:49 PM
Patrick Mours (pmoursnv) added a comment to D14630: Cycles Hydra: Add support for "stageMetersPerUnit" render setting.

It depends on what the USD exporter exported and assumes the Hydra viewport sets this render setting (or the user has to manually):
The Blender USD exporter currently exports USD with "metersPerUnit" set to 1, so assuming the app (Houdini does this for example) or user set the "stageMetersPerUnit" render setting to 1 (since it defaults to 0.01), this change should have no effect.
The Blender USD exporter from the "universal-scene-description" branch on the other hand has an option to convert to centimeters, in which case transforms are scaled by 100 and the USD has the "metersPerUnit" metadata set to 0.01. There those transforms need to be scaled with that 0.01 again to get back to the original values, which is what this change does. This way light radius and intensity should work as expected, as USD does not scale them by the transform and the Blender USD exporter dumps them to USD as is.

Apr 19 2022, 2:19 PM

Apr 13 2022

Patrick Mours (pmoursnv) committed rBc31b89e76e0d: Cycles: Add support for "stageMetersPerUnit" Hydra render setting (authored by Patrick Mours (pmoursnv)).
Cycles: Add support for "stageMetersPerUnit" Hydra render setting
Apr 13 2022, 12:25 PM
Patrick Mours (pmoursnv) closed D14630: Cycles Hydra: Add support for "stageMetersPerUnit" render setting.
Apr 13 2022, 12:25 PM

Apr 12 2022

Patrick Mours (pmoursnv) requested review of D14630: Cycles Hydra: Add support for "stageMetersPerUnit" render setting.
Apr 12 2022, 7:59 PM
Patrick Mours (pmoursnv) added a comment to D14594: Cycles Hydra: support for building against Houdini.

A few more necessary changes I had missed mentioning the first time around, with these it now builds successfully on Windows too.

Apr 12 2022, 7:35 PM

Apr 8 2022

Patrick Mours (pmoursnv) added a comment to T96731: Cycles Hydra Render Delegate.

Awesome! There is a bit more necessary to get it running with Houdini on Windows it seems. I posted some suggestions to D14594 that made it work for me.

Apr 8 2022, 6:30 PM · Cycles, Render & Cycles
Patrick Mours (pmoursnv) awarded D14594: Cycles Hydra: support for building against Houdini a Love token.
Apr 8 2022, 6:05 PM
Patrick Mours (pmoursnv) awarded rB0de0950ad5d1: Cycles: various Linux build fixes related to Hydra render delegate a Love token.
Apr 8 2022, 12:15 PM

Apr 5 2022

Patrick Mours (pmoursnv) committed rBe51368728815: Cycles: Fix a few type casting warnings (authored by Patrick Mours (pmoursnv)).
Cycles: Fix a few type casting warnings
Apr 5 2022, 6:10 PM
Patrick Mours (pmoursnv) closed D14558: Cycles: Fix a few type casting warnings.
Apr 5 2022, 6:10 PM
Patrick Mours (pmoursnv) requested review of D14558: Cycles: Fix a few type casting warnings.
Apr 5 2022, 5:43 PM
John Cox (johnedwardcox) awarded rBf60cffad38d1: Cycles: Use USD dependencies when building Hydra render delegate a Love token.
Apr 5 2022, 5:35 PM
John Cox (johnedwardcox) awarded rBf60cffad38d1: Cycles: Use USD dependencies when building Hydra render delegate a 100 token.
Apr 5 2022, 5:35 PM
Patrick Mours (pmoursnv) committed rBf60cffad38d1: Cycles: Use USD dependencies when building Hydra render delegate (authored by Patrick Mours (pmoursnv)).
Cycles: Use USD dependencies when building Hydra render delegate
Apr 5 2022, 5:25 PM
Patrick Mours (pmoursnv) closed D14523: Cycles: Use USD dependencies when building Hydra render delegate.
Apr 5 2022, 5:25 PM

Apr 4 2022

Patrick Mours (pmoursnv) added inline comments to D14523: Cycles: Use USD dependencies when building Hydra render delegate.
Apr 4 2022, 8:03 PM
Patrick Mours (pmoursnv) added a comment to D14537: WIP: USD cycles procedural.

It's doable, I have a prototype lying around that needs some clean up. It instantiates the Cycles Hydra render delegate (with an existing Cycles session: https://developer.blender.org/diffusion/B/browse/master/intern/cycles/hydra/render_delegate.h$17), loads the USD stage and pushes it into the render delegate to sync. That takes care of setting up the Cycles scene and can then render as usual.
Does require a USD build with Hydra though of course, so if we want to make building the Cycles standalone with USD support using the precompiled libraries only possible, would need to update the precompiled USD to also include the Hydra component.

Apr 4 2022, 1:04 PM · USD, Render & Cycles, Cycles

Apr 1 2022

Patrick Mours (pmoursnv) added inline comments to D14523: Cycles: Use USD dependencies when building Hydra render delegate.
Apr 1 2022, 11:12 PM
Patrick Mours (pmoursnv) added a revision to T96731: Cycles Hydra Render Delegate: D14523: Cycles: Use USD dependencies when building Hydra render delegate.
Apr 1 2022, 11:01 PM · Cycles, Render & Cycles
Patrick Mours (pmoursnv) requested review of D14523: Cycles: Use USD dependencies when building Hydra render delegate.
Apr 1 2022, 11:01 PM

Mar 25 2022

kursad k (kursadk) awarded D14398: Cycles: Add Hydra render delegate a Burninate token.
Mar 25 2022, 4:10 PM · Render & Cycles

Mar 24 2022

Patrick Mours (pmoursnv) added a comment to T96731: Cycles Hydra Render Delegate.

Right, the build process could use some improvement. Setting USD_ROOT, TBB_ROOT(_DIR), OPENVDB_ROOT(_DIR) etc. works for the moment though (most importantly USD_ROOT and OPENVDB_ROOT, the others are used in mostly ABI compatible ways, so doesn't matter as much if linking against the different version in the precompiled libraries). Can also just disable OpenVDB support to not worry about it at all with WITH_OPENVDB=0. To get a USD build working that was built with Python, currently have to set a USD_PYTHON_LIBRARIES CMake variable with the additional libs to link (e.g. USD_PYTHON_LIBRARIES=boost_python37-vc141-mt-x64-1_68;python37 on Windows for Python 3.7).

Mar 24 2022, 4:22 PM · Cycles, Render & Cycles
Aditia A. Pratama (aditiapratama) awarded D14398: Cycles: Add Hydra render delegate a Love token.
Mar 24 2022, 6:50 AM · Render & Cycles
Alaska (Alaska) awarded rBd350976ba06d: Cycles: Add Hydra render delegate a Love token.
Mar 24 2022, 1:54 AM

Mar 23 2022

Patrick Mours (pmoursnv) committed rB5d38b13e61ff: CMake: Rename "USD_LIBRARY_PREFIX" to "PXR_LIB_PREFIX" for consistency (authored by Patrick Mours (pmoursnv)).
CMake: Rename "USD_LIBRARY_PREFIX" to "PXR_LIB_PREFIX" for consistency
Mar 23 2022, 4:52 PM
Patrick Mours (pmoursnv) committed rBd350976ba06d: Cycles: Add Hydra render delegate (authored by Patrick Mours (pmoursnv)).
Cycles: Add Hydra render delegate
Mar 23 2022, 4:40 PM
Patrick Mours (pmoursnv) closed D14398: Cycles: Add Hydra render delegate.
Mar 23 2022, 4:40 PM · Render & Cycles

Mar 22 2022

Patrick Mours (pmoursnv) updated the summary of D14398: Cycles: Add Hydra render delegate.
Mar 22 2022, 2:28 PM · Render & Cycles
Patrick Mours (pmoursnv) updated the diff for D14398: Cycles: Add Hydra render delegate.

Reverted unnecessary changes to "FindUSD.cmake" in favor of using "USD_ROOT" to specifc location

Mar 22 2022, 2:26 PM · Render & Cycles
Patrick Mours (pmoursnv) added inline comments to D14398: Cycles: Add Hydra render delegate.
Mar 22 2022, 2:22 PM · Render & Cycles

Mar 21 2022

Patrick Mours (pmoursnv) added inline comments to D14398: Cycles: Add Hydra render delegate.
Mar 21 2022, 5:18 PM · Render & Cycles
Patrick Mours (pmoursnv) updated the summary of D14398: Cycles: Add Hydra render delegate.
Mar 21 2022, 5:03 PM · Render & Cycles
Patrick Mours (pmoursnv) updated the diff for D14398: Cycles: Add Hydra render delegate.

Fixed Windows build to use existing WITH_WINDOWS_FIND_MODULES CMake option.

Mar 21 2022, 5:02 PM · Render & Cycles
Manuel Grad (manitwo) awarded D14398: Cycles: Add Hydra render delegate a Love token.
Mar 21 2022, 2:33 PM · Render & Cycles
Raimund Klink (Raimund58) awarded D14398: Cycles: Add Hydra render delegate a Love token.
Mar 21 2022, 2:07 PM · Render & Cycles
William Reynish (billreynish) awarded D14398: Cycles: Add Hydra render delegate a Love token.
Mar 21 2022, 2:01 PM · Render & Cycles
Brecht Van Lommel (brecht) awarded D14398: Cycles: Add Hydra render delegate a Love token.
Mar 21 2022, 1:58 PM · Render & Cycles
Patrick Mours (pmoursnv) updated the summary of D14398: Cycles: Add Hydra render delegate.
Mar 21 2022, 1:54 PM · Render & Cycles
Patrick Mours (pmoursnv) updated the summary of D14398: Cycles: Add Hydra render delegate.
Mar 21 2022, 1:37 PM · Render & Cycles
Patrick Mours (pmoursnv) updated the summary of D14398: Cycles: Add Hydra render delegate.
Mar 21 2022, 1:33 PM · Render & Cycles
Patrick Mours (pmoursnv) requested review of D14398: Cycles: Add Hydra render delegate.
Mar 21 2022, 1:23 PM · Render & Cycles

Mar 14 2022

Patrick Mours (pmoursnv) closed T91849: Support for Blender 3.0 as Resolved.
Mar 14 2022, 4:19 PM · Blender Open Data

Mar 10 2022

Patrick Mours (pmoursnv) accepted D14300: Fix T93710: Artifacts denoising hi-res images using OPtiX.

Thanks for taking care of doing the patch!

Mar 10 2022, 4:17 PM

Mar 1 2022

Patrick Mours (pmoursnv) added a comment to T95836: Viewport performance slows down when using OptiX Denoiser on Windows machine with dual GPUs..

Good idea, though would need a bit of refactoring. Probably something like (pseudo code):

Mar 1 2022, 2:33 PM · Render & Cycles, Cycles, BF Blender
Patrick Mours (pmoursnv) updated subscribers of T95836: Viewport performance slows down when using OptiX Denoiser on Windows machine with dual GPUs..

This is not Windows specific and is related to how tiling and multi GPU is currently implemented in Cycles X:

Mar 1 2022, 1:09 PM · Render & Cycles, Cycles, BF Blender

Feb 24 2022

Patrick Mours (pmoursnv) added a comment to T93710: Renderings vom 2.9 LTS produce different results than 3.0.

Like I said, you need at least 511.79 on Windows, 511.65 is too old.

Feb 24 2022, 2:37 PM · Render & Cycles, BF Blender

Feb 23 2022

Patrick Mours (pmoursnv) added a comment to T93710: Renderings vom 2.9 LTS produce different results than 3.0.

I was able to get rid of the artifacts with just this patch and a driver that has the driver-side fixes (they made it into the last r510 release, so need at least Windows 511.79 or Linux 510.54):

Feb 23 2022, 4:38 PM · Render & Cycles, BF Blender

Feb 18 2022

Patrick Mours (pmoursnv) added a comment to T95814: OptiX Denoiser created a horizontal yellow ribbon.

Sure! But good point with the driver requirements ... Maybe better to reimplement the helper function or just copy those contents of "optix_denoiser_tiling.h" into the code and fix it (that particular file is under BSD 3-clause license, so no problem).

Feb 18 2022, 3:40 PM · Render & Cycles, BF Blender
Patrick Mours (pmoursnv) added a comment to T95814: OptiX Denoiser created a horizontal yellow ribbon.

Yeah, this looks like a duplicate of T93710. That hasn't been fully fixed yet (the driver part is fixed in the last r510 drivers, but the same integer overflow problem is also present in the optixUtilDenoiserInvokeTiled helper function, which needs a SDK update to fix or will have to copy the function implementation into Cycles source code).

Feb 18 2022, 2:58 PM · Render & Cycles, BF Blender

Feb 8 2022

Davide Caprioli (DaveDeer) awarded rB8393ccd07634: Cycles: Add OptiX temporal denoising support a Love token.
Feb 8 2022, 10:17 PM

Feb 1 2022

SDPM Esare (SDPM) awarded D11442: Cycles X: Add OptiX temporal denoising support a Love token.
Feb 1 2022, 1:56 PM

Jan 31 2022

Patrick Mours (pmoursnv) added a comment to T91849: Support for Blender 3.0.

Are there any news or is there help needed for this?

Jan 31 2022, 12:43 PM · Blender Open Data

Jan 26 2022

Sergey Morozov (smorozov) awarded rB8393ccd07634: Cycles: Add OptiX temporal denoising support a Love token.
Jan 26 2022, 9:14 AM

Jan 10 2022

Gilberto Rodrigues (gilberto_rodrigues) awarded rB8393ccd07634: Cycles: Add OptiX temporal denoising support a Love token.
Jan 10 2022, 1:43 PM

Jan 7 2022

Patrick Mours (pmoursnv) closed T94711: Blender crashes with on Apple M1 when using Cycles with GPU Compute since rB8393ccd07634 as Resolved.

Fixed in rBefe3d60a2c8306aefd41bc304548da35b67c252c.

Jan 7 2022, 5:30 PM · Render & Cycles, Platform: macOS, BF Blender
Patrick Mours (pmoursnv) abandoned D13761: Fix T94711: Blender crashes with on Apple M1 when using Cycles with GPU Compute.

Thanks! Closing as this was now committed in rBefe3d60a2c8306aefd41bc304548da35b67c252c.

Jan 7 2022, 5:29 PM · Platform: macOS, Render & Cycles
Patrick Mours (pmoursnv) closed D13767: Cycles crashes during initialization on Mac Metal GPU.
Jan 7 2022, 5:22 PM · BF Blender (3.1)
Patrick Mours (pmoursnv) added a comment to D13767: Cycles crashes during initialization on Mac Metal GPU.

Ah, didn't see D13763, in that case probably easier if you just land that one instead of this one =)

Jan 7 2022, 5:16 PM · BF Blender (3.1)
Patrick Mours (pmoursnv) accepted D13767: Cycles crashes during initialization on Mac Metal GPU.

Thanks! Will commit in a sec.

Jan 7 2022, 5:09 PM · BF Blender (3.1)

Jan 6 2022

Patrick Mours (pmoursnv) added a comment to D13742: Fix: MSVC2017 build error.

Was looking at the same thing and problem appeared to be the reports_lock field actually: The SpinLock type was typedef'd to volatile unsigned int, since the bf_editor_render project does not define WITH_TBB (which otherwise causes SpinLock to be typedef'd to uint32_t and is actually what should have been used here). And when VC++ 2017 sees a volatile in a type definition it no longer considers it trivial, hence why the check failed.

Jan 6 2022, 1:19 PM

Jan 5 2022

Alaska (Alaska) awarded rB8393ccd07634: Cycles: Add OptiX temporal denoising support a Love token.
Jan 5 2022, 9:01 PM
Patrick Mours (pmoursnv) committed rB8393ccd07634: Cycles: Add OptiX temporal denoising support (authored by Patrick Mours (pmoursnv)).
Cycles: Add OptiX temporal denoising support
Jan 5 2022, 3:59 PM
Patrick Mours (pmoursnv) closed D11442: Cycles X: Add OptiX temporal denoising support.
Jan 5 2022, 3:59 PM

Jan 4 2022

Patrick Mours (pmoursnv) updated the diff for D11442: Cycles X: Add OptiX temporal denoising support.

Fixed temporal denoising mode not actually getting enabled (since first frame is denoised with the normal denoiser still and it failed to switch on the second frame onwards). Whoops.

Jan 4 2022, 9:41 PM

Jan 3 2022

Vitalii Vlasov (Fracture128) awarded D11442: Cycles X: Add OptiX temporal denoising support a Like token.
Jan 3 2022, 4:51 PM
Patrick Mours (pmoursnv) updated the diff for D11442: Cycles X: Add OptiX temporal denoising support.

Rebased and removed motion pass addition (hadn't realized what this entails). It's certainly fair to just note that both Denoising Data and Vector passes need to be enabled for the operator to work. If this is not the case it will report an error now ("Could not find a render layer containing denoising data and motion vector passes").

Jan 3 2022, 4:45 PM
Patrick Mours (pmoursnv) committed rBca143fafa674: Cleanup: Silence "integer conversion resulted in a change of sign" warning in… (authored by Patrick Mours (pmoursnv)).
Cleanup: Silence "integer conversion resulted in a change of sign" warning in…
Jan 3 2022, 4:29 PM

Dec 13 2021

Raimund Klink (Raimund58) awarded rBe14f8c2dd765: Cycles: Reintroduce device-only memory handling that got lost in Cycles X merge a Love token.
Dec 13 2021, 10:56 AM

Dec 10 2021

Gilberto Rodrigues (gilberto_rodrigues) awarded D11442: Cycles X: Add OptiX temporal denoising support a Love token.
Dec 10 2021, 9:32 AM

Dec 9 2021

Alexey Adamitsky (AlexeyAdamitsky) awarded D11442: Cycles X: Add OptiX temporal denoising support a Love token.
Dec 9 2021, 2:02 PM
Patrick Mours (pmoursnv) changed the status of T93710: Renderings vom 2.9 LTS produce different results than 3.0 from Needs Information from Developers to Confirmed.

So the problem here is a traditional case of integer overflow. What is happening is that for denoising, Cycles loads the entire huge image into a GPU buffer. Since rB17665494186816cebb9e8304199e40f9ee033990 this now theoretically works with the OptiX denoiser by splitting up this image buffer into smaller tiles that are denoised separately. But there is a catch: the width, height and rowStrideInBytes values passed to the denoiser are 32-bit integers and unfortunately there is code to calculate the size of the entire image by multiplying height (or a y coordinate close to that) and rowStrideInBytes. In this case the result is larger than what a 32-bit integer can hold and therefore overflows, causing the denoiser to read/write data at bogus locations, which shows up as artifacts. rowStrideInBytes is affected by the width of the image and how many render passes are active, hence why the problem goes away if you reduce the width or disable enough render passes (including hidden render passes that one does not select in the GUI, e.g. enabling auto tiling adds a hidden render pass, adaptive sampling adds two hidden render passes). To fix this the calculation will need to be done in 64-bit (by casting to 64-bit before multiplying).

Dec 9 2021, 12:37 PM · Render & Cycles, BF Blender

Dec 8 2021

Christian Friedrich (rbx775) awarded D11442: Cycles X: Add OptiX temporal denoising support a Love token.
Dec 8 2021, 11:23 PM

Dec 7 2021

Patrick Mours (pmoursnv) closed D13488: Cycles: Reintroduce device-only memory handling that got lost in Cycles X merge.
Dec 7 2021, 6:50 PM
Patrick Mours (pmoursnv) committed rBe14f8c2dd765: Cycles: Reintroduce device-only memory handling that got lost in Cycles X merge (authored by Patrick Mours (pmoursnv)).
Cycles: Reintroduce device-only memory handling that got lost in Cycles X merge
Dec 7 2021, 6:50 PM
Rincewind (Rincewind3D) awarded D13488: Cycles: Reintroduce device-only memory handling that got lost in Cycles X merge a Love token.
Dec 7 2021, 11:46 AM
Patrick Mours (pmoursnv) added a comment to T93620: Cycles X broke OptiX memory pooling via NVLink.

The OPTIX_ERROR_INVALID_VALUE you are seeing is happening because of an out of memory (you can verify this is the log, with "--debug-cycles"). Likely this is the case because BVH builds are happening in parallel, which quickly exhausts available memory because of temporary build memory required (and has some additional known quirks when memory pooling is active), rather than serialized which does not suffer from that problem. That was fixed before (and is in 2.93), but the fix got lost in the Cycles X merge (and thus is not in 3.0), hence why 3.0 behaves differently. The rest of the pooled memory implementation has not changed.

Dec 7 2021, 11:42 AM · Render & Cycles, Cycles, BF Blender
Patrick Mours (pmoursnv) added a comment to T93567: Optix Artefacts with motion blur (Alembic?).

I think this is not enough to continue investigating currently, without being able to reproduce. And since it's not happening with buildbot builds either it's probably not as critical. So I'd lean towards closing for now and reopen later again if it becomes a bigger problem.

Dec 7 2021, 11:36 AM · BF Blender

Dec 6 2021

Patrick Mours (pmoursnv) added a revision to T93620: Cycles X broke OptiX memory pooling via NVLink: D13488: Cycles: Reintroduce device-only memory handling that got lost in Cycles X merge.
Dec 6 2021, 3:31 PM · Render & Cycles, Cycles, BF Blender
Patrick Mours (pmoursnv) requested review of D13488: Cycles: Reintroduce device-only memory handling that got lost in Cycles X merge.
Dec 6 2021, 3:31 PM

Dec 3 2021

Patrick Mours (pmoursnv) added a comment to T93567: Optix Artefacts with motion blur (Alembic?).

Would be interesting to know what happens on 495.44, since @Philipp Oeser (lichtwerk) didn't reproduce there, and it's a newer stable release from the same branch as 495.29.05 (which is considered a beta driver).

Dec 3 2021, 5:20 PM · BF Blender