Skip to content

Conversation

@T-K-233
Copy link
Contributor

@T-K-233 T-K-233 commented Jul 1, 2025

//Before Merge//

I would like to know if this is the proper way to implement this functionality. Please let me know if there are better ways to structure the code.

Description

This PR creates SymmetryCfg and SymmetryTermCfg config classes that can be used to set symmetric configuration to the RSL-RL framework. An example data augmentation function, symmetry_data_augmentation_function, is also provided to implement corresponding symmetry terms.

This PR answers Issue #2835.

Type of change

  • 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

Screenshots

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

Checklist

  • I have run the pre-commit checks 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

@kellyguo11
Copy link
Contributor

Thanks for the PR! We will work on putting out an example setup with symmetry soon.

@T-K-233
Copy link
Contributor Author

T-K-233 commented Jul 12, 2025

Sounds good! Please let me know if I can help on anything

@Mayankm96
Copy link
Contributor

@T-K-233 I merged the symmetry MR with examples for multiple environments. Please feel free to adapt this MR to that. Let us know if you have questions.

@T-K-233 T-K-233 closed this Sep 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants