[Bug Report] Fresnel Texture Custom n, k Values Are Limited to 0 to 1 Range
The fresnel texture, when set to use custom n and k values only allows colours with individual values in a 0 to 1 range. Many n and k go beyond this range.
Steps to reproduce:
Open the luxcore material editor in blender 4.3. Add a fresnel texture node. Set it to custom n and k values. Try to type a value larger than 1 into one of the colour slots.
Please provide a simple test scene as a .blend-file:
I tried to provide a .blend file but that type of file is not supported when trying to upload it here.
Screenshots or rendered images:
Error message from Blender console:
Operating System: Windows Blender version: 4.3.2 BlendLuxCore version: 2.10 Pyluxcore version: CPU and GPU model:
Open ToDo items on this issue:
- [ ] Update the default value for custom n,k to accurately match one of the other presets, e.g. Aluminium
After a bit of a recap-session, because I felt like hearing about this before, I discovered that indeed I was trying to deal with this 3 years ago:
https://forums.luxcorerender.org/viewtopic.php?p=33839#p33839
At the time there was no active maintainer, and I did not yet have rights, so it was never included, and I don't have that version of the code anymore... Should be easy enough to recreate though.
I re-implemented as shown in the previously linked forums post:
https://github.com/LuxCoreRender/BlendLuxCore/commit/04ca6ac4fd2c99d0dc8f6b94792e1c5fb321dead
The changes are in for_v2.11 branch. I'm not automatically building a "latest" release at the moment, so if you need it quickly and know how to edit the code yourself I suggest to do that for the moment.
Leaving this issue open as a reminder to consider better default values.
Thanks! Is there any way to track the goals, progress and potential release date of future updates?
Nothing concrete yet. v2.10 has only recently been released, we need to set up the plan for 2.11, currently happening in the background. Howetuft already created the discussions category, but there is no content yet: https://github.com/orgs/LuxCoreRender/discussions/categories/off-we-go-v2-11
Leaving this issue open as a reminder to consider better default values.
I created a new issue to track default values that need to change #1014
Shall we close this issue now?