Falcor icon indicating copy to clipboard operation
Falcor copied to clipboard

Bugfix: locale float to_string

Open steimich96 opened this issue 1 year ago • 1 comments

This is fixes a bug that leads to a compile error when the local system uses a comma (",") instead of dot (".") as decimal-point (e.g., in German).

Scene.cpp: defines.add("SCENE_DIFFUSE_ALBEDO_MULTIPLIER", std::to_string(mRenderSettings.diffuseAlbedoMultiplier));

StandardMaterial.slang: d.diffuse = saturate(d.diffuse * SCENE_DIFFUSE_ALBEDO_MULTIPLIER);

Example: std::cout << std::to_string(mRenderSettings.diffuseAlbedoMultiplier) << " vs. " << fmt::format("{:f}", RenderSettings.diffuseAlbedoMultiplier) << std::endl;

Output: 1,000000 vs. 1.000000

steimich96 avatar Feb 19 '24 12:02 steimich96

Do not merge yet. Found another one:

defines.add("MIS_POWER_EXPONENT", std::to_string(misPowerExponent));

steimich96 avatar Feb 19 '24 12:02 steimich96