Ray molenkamp LazyDodo
  • Joined on 2014-10-02
Ray molenkamp commented on issue blender/blender#124908 2024-07-18 17:05:33 +02:00
Some file formats save very slowly over fast VPN link

Didn't test all formats but, saving the default cube at 1080p

Ray molenkamp commented on issue blender/blender#124908 2024-07-18 15:53:50 +02:00
Some file formats save very slowly over fast VPN link

Someone will have to take a closer look, gut feeling says It's likely a latency/iops related issue, I suspect the fast formats likely do fewer large writes while the slower formats do frequent…

Ray molenkamp commented on issue blender/blender#124833 2024-07-18 06:13:12 +02:00
Windows installers (both msi and msix) and Portable for Blender 4.2 not opening
  1. No idea, this could be diagnosed by running msiexec /i blender-4.2.0-windows-x64.msi /l* blender.log and looking at the generated blender.log

2+3. The store version will use a sandboxed…

Ray molenkamp commented on pull request blender/blender#124916 2024-07-17 22:15:21 +02:00
CMake: Modernize the optional TBB dependency

I'd probably place just the add_definitions(-DNOMINMAX) back where they went missing but use the new target for everything else, as to not upset things too much and deal with a min/max cleanup…

Ray molenkamp commented on pull request blender/blender#124916 2024-07-17 22:11:28 +02:00
CMake: Modernize the optional TBB dependency
  • intern/mikktspace seems fine to change
  • intern/mantaflow doesn't use TBB just does the work so the headers from extern/mantaflow don't throw a fit
  • extern/mantaflow uses -DTBB=1
Ray molenkamp commented on pull request blender/blender#124916 2024-07-17 20:41:12 +02:00
CMake: Modernize the optional TBB dependency

lgtm, then again, i took the initial stab at it so i'm hardly unbiased here, we should probably have someone from core look at this. Maybe @Sergey or @mont29 be interested in reviewing this?

Ray molenkamp commented on pull request blender/blender#124916 2024-07-17 20:41:11 +02:00
CMake: Modernize the optional TBB dependency

-DNOMINMAX went missing here, which may come back to haunt you, my plan was to add this define to the platform flags so all files would have it, but have not done so since some warns popped up about double defines in places that also tried dealing with this problem themselves, which i had no time to resolve just yet.

Ray molenkamp commented on pull request blender/blender#124916 2024-07-17 20:41:10 +02:00
CMake: Modernize the optional TBB dependency

can't this just depend on bf::dependencies::optional::tbb ? ideally there should be no more instances of ${TBB_INCLUDE_DIRS} and ${TBB_LIBRARIES} through out our cmakefiles, except for the file that defines the ``bf::dependencies::optional::tbb` target

Ray molenkamp commented on issue blender/blender#124173 2024-07-17 06:30:56 +02:00
Grease Pencil: SVG export creates invalid xml

the PUGIXML_HEADER_ONLY doesn't work according to sebastian, we'll need #define PUGIXML_API __attribute__((visibility("hidden"))) but be sure to put it in a guard as not to trip up MSVC

Ray molenkamp commented on issue blender/blender#124173 2024-07-16 16:01:34 +02:00
Grease Pencil: SVG export creates invalid xml

some quick notes from the chat

`nm -D ./blender

Ray molenkamp commented on pull request blender/blender#121050 2024-07-12 00:13:43 +02:00
Cycles: Linux Support for HIP-RT

with BITCODE=OFF it appears to build fine with no further python issues, it warns about our cuda version

         -- CUDA SDK install folder found: C:\Program Files\NVIDIA GPU…
Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 22:54:09 +02:00
Cycles: Linux Support for HIP-RT

I'm afraid not, given we also build with BITCODE=ON it'll still call bakeKernel and run into python issues there.

Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 22:40:24 +02:00
Cycles: Linux Support for HIP-RT

Windows and linux are fundamentally not all that different, except for the make helper, couple of pointers here though:

  1. we build the deps on vs 2019 - v16.11.26 , it has seen no other more…
Ray molenkamp suggested changes for blender/blender#121050 2024-07-11 17:33:14 +02:00
Cycles: Linux Support for HIP-RT
Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 17:33:13 +02:00
Cycles: Linux Support for HIP-RT

This isn't used anywhere

Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 17:33:12 +02:00
Cycles: Linux Support for HIP-RT

Projects not building for Debug is generally handled in the main CMakeLists.txt

Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 17:33:11 +02:00
Cycles: Linux Support for HIP-RT

The marking of the script as executable can be handled using the PATCH_COMMAND as it only needs to be executed once.

Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 17:33:10 +02:00
Cycles: Linux Support for HIP-RT

by setting just CONFIGURE_COMMAND and not BUILD_COMMAND cmake will just call make to build the project which is problematic since the PLATFORM_ALT_GENERATOR is set to ninja.

Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 17:33:09 +02:00
Cycles: Linux Support for HIP-RT

Only a single after_install is allowed, these two will need to be merge or cmake will error out.

Ray molenkamp commented on pull request blender/blender#121050 2024-07-11 17:33:07 +02:00
Cycles: Linux Support for HIP-RT

This doesn't work on windows, it requires set PYTHON_BIN=${PYTHON_BINARY} this probably be fixed by sticking it in the HIPRT_WORKAROUND section above however since that has been moved to a patch step and....