Commit c37ccce8 authored by rwiltz's avatar rwiltz Committed by Kelly Guo

Removes xr rendering mode (#388)

Remove xr rendering mode since balanced and quality work well for xr
<!--
Thank you for your interest in sending a pull request. Please make sure
to check the contribution guidelines.

Link:
https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html
-->

Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context.
List any dependencies that are required for this change.

Fixes # (issue)

<!-- As a practice, it is recommended to open an issue to have
discussions on the proposed pull request.
This makes it easier for the community to keep track of what is being
developed or added, and if a given feature
is demanded by more than one party. -->

<!-- As you go through the list, delete the ones that are not
applicable. -->

- Bug fix (non-breaking change which fixes an issue)
- New feature (non-breaking change which adds functionality)
- Breaking change (fix or feature that would cause existing
functionality to not work as expected)
- This change requires a documentation update

Please attach before and after screenshots of the change if applicable.

<!--
Example:

| Before | After |
| ------ | ----- |
| _gif/png before_ | _gif/png after_ |

To upload images to a PR -- simply drag and drop an image while in edit
mode and it should upload the image directly. You can then paste that
source into the above before/after sections.
-->

- [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have updated the changelog and the corresponding version in the
extension's `config/extension.toml` file
- [ ] I have added my name to the `CONTRIBUTORS.md` or my name already
exists there

<!--
As you go through the checklist above, you can mark something as done by
putting an x character in it

For example,
- [x] I have done this task
- [ ] I have not done this task
-->
parent b68cbe2f
rtx.translucency.enabled = true
rtx.reflections.enabled = true
rtx.reflections.denoiser.enabled = true
rtx.directLighting.sampledLighting.denoisingTechnique = 5
rtx.directLighting.sampledLighting.enabled = true
rtx.sceneDb.ambientLightIntensity = 1.0
rtx.shadows.enabled = true
rtx.indirectDiffuse.enabled = true
rtx.indirectDiffuse.denoiser.enabled = true
rtx.domeLight.upperLowerStrategy = 4
rtx.ambientOcclusion.enabled = true
rtx.ambientOcclusion.denoiserMode = 0
rtx.raytracing.subpixel.mode = 1
rtx.raytracing.cached.enabled = true
# DLSS frame gen does not yet support tiled camera well
rtx-transient.dlssg.enabled = false
rtx-transient.dldenoiser.enabled = true
# Set the DLSS model
rtx.post.dlss.execMode = 2 # can be 0 (Performance), 1 (Balanced), 2 (Quality), or 3 (Auto)
# Avoids replicator warning
rtx.pathtracing.maxSamplesPerLaunch = 1000000
# Avoids silent trimming of tiles
rtx.viewTile.limit = 1000000
...@@ -391,6 +391,15 @@ Added ...@@ -391,6 +391,15 @@ Added
* Added :meth:`~isaaclab.envs.mdp.observations.body_projected_gravity_b` * Added :meth:`~isaaclab.envs.mdp.observations.body_projected_gravity_b`
0.37.1 (2025-05-05)
~~~~~~~~~~~~~~~~~~~~
Changed
^^^^^^^
* Removed xr rendering mode.
0.37.0 (2025-04-24) 0.37.0 (2025-04-24)
~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~
......
...@@ -321,10 +321,10 @@ class AppLauncher: ...@@ -321,10 +321,10 @@ class AppLauncher:
"--rendering_mode", "--rendering_mode",
type=str, type=str,
action=ExplicitAction, action=ExplicitAction,
choices={"performance", "balanced", "quality", "xr"}, choices={"performance", "balanced", "quality"},
help=( help=(
"Sets the rendering mode. Preset settings files can be found in apps/rendering_modes." "Sets the rendering mode. Preset settings files can be found in apps/rendering_modes."
' Can be "performance", "balanced", "quality", or "xr".' ' Can be "performance", "balanced", or "quality".'
" Individual settings can be overwritten by using the RenderCfg class." " Individual settings can be overwritten by using the RenderCfg class."
), ),
) )
...@@ -860,14 +860,6 @@ class AppLauncher: ...@@ -860,14 +860,6 @@ class AppLauncher:
if rendering_mode is None: if rendering_mode is None:
rendering_mode = "balanced" rendering_mode = "balanced"
rendering_mode_explicitly_passed = launcher_args.pop("rendering_mode_explicit", False)
if self._xr and not rendering_mode_explicitly_passed:
# If no rendering mode is specified, default to the xr mode if we are running in XR
rendering_mode = "xr"
# Overwrite for downstream consumers
launcher_args["rendering_mode"] = "xr"
# parse preset file # parse preset file
repo_path = os.path.join(carb.tokens.get_tokens_interface().resolve("${app}"), "..") repo_path = os.path.join(carb.tokens.get_tokens_interface().resolve("${app}"), "..")
preset_filename = os.path.join(repo_path, f"apps/rendering_modes/{rendering_mode}.kit") preset_filename = os.path.join(repo_path, f"apps/rendering_modes/{rendering_mode}.kit")
......
...@@ -257,7 +257,7 @@ class RenderCfg: ...@@ -257,7 +257,7 @@ class RenderCfg:
rtx.translucency.enabled: False # .kit rtx.translucency.enabled: False # .kit
rtx_translucency_enabled: False # python""" rtx_translucency_enabled: False # python"""
rendering_mode: Literal["performance", "balanced", "quality", "xr"] | None = None rendering_mode: Literal["performance", "balanced", "quality"] | None = None
"""Sets the rendering mode. Behaves the same as the CLI arg '--rendering_mode'""" """Sets the rendering mode. Behaves the same as the CLI arg '--rendering_mode'"""
......
...@@ -303,7 +303,7 @@ class SimulationContext(_SimulationContext): ...@@ -303,7 +303,7 @@ class SimulationContext(_SimulationContext):
rendering_mode = self.cfg.render.rendering_mode rendering_mode = self.cfg.render.rendering_mode
if rendering_mode is not None: if rendering_mode is not None:
# check if preset is supported # check if preset is supported
supported_rendering_modes = ["performance", "balanced", "quality", "xr"] supported_rendering_modes = ["performance", "balanced", "quality"]
if rendering_mode not in supported_rendering_modes: if rendering_mode not in supported_rendering_modes:
raise ValueError( raise ValueError(
f"RenderCfg rendering mode '{rendering_mode}' not in supported modes {supported_rendering_modes}." f"RenderCfg rendering mode '{rendering_mode}' not in supported modes {supported_rendering_modes}."
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment