-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Restore RT2 RenderCfg fields removed in PhysicsManager refactor #5167
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -151,6 +151,88 @@ class RenderCfg: | |||||
| This is set by the variable: ``/rtx/domeLight/upperLowerStrategy``. | ||||||
| """ | ||||||
|
|
||||||
| max_bounces: int | None = None | ||||||
| """Maximum number of ray bounces for path tracing (RT2). Default is 2. | ||||||
|
|
||||||
| For global illumination (indirect diffuse), this should be at least 3. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/rtpt/maxBounces``. | ||||||
| """ | ||||||
|
|
||||||
| split_glass: bool | None = None | ||||||
| """Enables separate glass ray splitting for improved glass rendering (RT2). Default is False. | ||||||
|
|
||||||
| Enabling this can reduce noise on glass materials at the cost of performance. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/rtpt/splitGlass``. | ||||||
| """ | ||||||
|
|
||||||
| split_clearcoat: bool | None = None | ||||||
| """Enables separate clearcoat ray splitting (RT2). Default is False. | ||||||
|
|
||||||
| Enabling this can reduce noise on clearcoat materials at the cost of performance. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/rtpt/splitClearcoat``. | ||||||
| """ | ||||||
|
|
||||||
| split_rough_reflection: bool | None = None | ||||||
| """Enables separate rough reflection ray splitting (RT2). Default is True. | ||||||
|
|
||||||
| Enabling this can reduce noise on rough reflective materials at the cost of performance. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/rtpt/splitRoughReflection``. | ||||||
| """ | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The original PR #4142 had this as "Default is False", and This should be restored to match the original:
Suggested change
|
||||||
|
|
||||||
| ambient_light_intensity: float | None = None | ||||||
| """Scene ambient light intensity. Default is 1.0. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/sceneDb/ambientLightIntensity``. | ||||||
| """ | ||||||
|
|
||||||
| ambient_occlusion_denoiser_mode: Literal[0, 1] | None = None | ||||||
| """Ambient occlusion denoiser mode. Default is 1. | ||||||
|
|
||||||
| Valid values are: | ||||||
|
|
||||||
| * 0: Higher quality denoising | ||||||
| * 1: Performance-oriented denoising | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/ambientOcclusion/denoiserMode``. | ||||||
| """ | ||||||
|
|
||||||
| subpixel_mode: Literal[0, 1] | None = None | ||||||
| """Raytracing subpixel mode. Default is 0. | ||||||
|
|
||||||
| Valid values are: | ||||||
|
|
||||||
| * 0: Performance mode | ||||||
| * 1: Quality mode (better anti-aliasing) | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/raytracing/subpixel/mode``. | ||||||
| """ | ||||||
|
|
||||||
| enable_cached_raytracing: bool | None = None | ||||||
| """Enables cached raytracing for improved performance. Default is True. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/raytracing/cached/enabled``. | ||||||
| """ | ||||||
|
|
||||||
| max_samples_per_launch: int | None = None | ||||||
| """Maximum samples per launch for path tracing. Default is 1000000. | ||||||
|
|
||||||
| This setting helps avoid replicator warnings when using large tile counts. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/pathtracing/maxSamplesPerLaunch``. | ||||||
| """ | ||||||
|
|
||||||
| view_tile_limit: int | None = None | ||||||
| """Maximum number of view tiles. Default is 1000000. | ||||||
|
|
||||||
| This setting helps avoid silent trimming of tiles. | ||||||
|
|
||||||
| This is set by the variable: ``/rtx/viewTile/limit``. | ||||||
| """ | ||||||
|
|
||||||
| carb_settings: dict[str, Any] | None = None | ||||||
| """A general dictionary for users to supply all carb rendering settings with native names. | ||||||
|
|
||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
split_rough_reflectionis documented as "Default is True", buttest_render_cfg_defaults(line 213) usesFalseas the expected default value under the# RT2 defaultscomment. Similarly,enable_cached_raytracingis documented as "Default is True" (line 215) while the test usesFalse(line 216). One of these is wrong — either the carb-setting system default changed, or the docstrings are incorrect. If the actual carb defaults areFalse, the docstrings should read "Default is False".