Lukas Stockner LukasStockner
  • Joined on 2013-12-26
Lukas Stockner commented on pull request blender/blender#123046 2024-06-16 15:55:06 +02:00
Cycles: add Central Cylindrical Panoramic camera model

This is only a small detail, but: I'd prefer to only have the radius in the UI. Perform the division here, then only handle u/v_min/max in the rest of the Cycles code.

Lukas Stockner commented on pull request blender/blender#123046 2024-06-16 15:55:05 +02:00
Cycles: add Central Cylindrical Panoramic camera model

The padding is needed for alignment reasons, this is all over the DNA files.

Lukas Stockner commented on pull request blender/blender#123046 2024-06-16 15:55:04 +02:00
Cycles: add Central Cylindrical Panoramic camera model

I think these should be hard limits, not UI limits? Same goes for u_max and radius (that one doesn't need an upper hard limit, but it shouldn't be negative).

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:56 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

"this object" -> "the center of the objects in this collection"

Lukas Stockner suggested changes for blender/blender#119869 2024-06-16 15:46:56 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

Overall the functionality seems reasonable, but there's a bunch of code comments. Also, I think the collection needs to be handled in DepsgraphRelationBuilder::build_camera as well.

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:55 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

Using FOREACH_COLLECTION_OBJECT_RECURSIVE_BEGIN would be cleaner here, and would avoid recursing explicitly.

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:54 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

This file is already C++, so I'd prefer to use the new float3 consistently instead of C-style arrays.

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:51 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

There's a bunch of duplication here compared to the focus-on-object case. I think we could just have two branches to find pos and then share the rest of the code?

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:50 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

No need to do this so explicitly, doing pos / count below should automatically perform the cast and apply to all components.

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:49 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

pos += transform_get_column(&dofmat, 3); should work fine here, no need for an intermediate variable or componentwise addition.

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:48 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

I think using coll.all_objects here removes the need to recurse over child collections, so this could just be done in blender_camera_focal_distance.

Lukas Stockner commented on pull request blender/blender#119869 2024-06-16 15:46:47 +02:00
Render: allow to select an entire collection to set the focal point in the Depth of Field settings in the Camera properties

Instead of hiding the UI elements here, I think it would be better to disable (gray out) the unavailable ones.

Lukas Stockner approved blender/blender#123267 2024-06-16 13:27:40 +02:00
Fix #123249: Orthographic DOF is influenced by near clip

Cycles also seems fine to me, but I don't know what the policy on compatibility is here. Personally, I think it's fine to just fix it.

Lukas Stockner commented on issue blender/blender#122990 2024-06-16 03:44:58 +02:00
Color management: Easy way to define white point

I've implemented an initial version of this in !123278, no support for color picker yet.

Lukas Stockner created pull request blender/blender#123278 2024-06-16 03:43:18 +02:00
Color management: Support white balance as part of the display transform
Lukas Stockner pushed to white-balance at LukasStockner/blender 2024-06-16 03:37:06 +02:00
3bf3638fd0 Color management: Support white balance as part of the display transform
Lukas Stockner created branch white-balance in LukasStockner/blender 2024-06-16 03:36:11 +02:00