Skip to content

Conversation

@Mayankm96
Copy link
Contributor

@Mayankm96 Mayankm96 commented Feb 20, 2025

Description

This MR adds a script to run training over a subset of environments. I have mostly checked this for rsl-rl library.

The script adds the commit tag as a run name allowing us to compare different versions of code nicely.

Type of change

  • New feature (non-breaking change which adds functionality)

Screenshots

Example tensorboard log:

./isaaclab.sh -p tools/run_train_envs.py --lib-name rsl_rl

image

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

Copy link
Collaborator

@jtigue-bdai jtigue-bdai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a pre-commit formatting is needed though

@jsmith-bdai
Copy link
Collaborator

This is awesome! We will likely re-use this for our internal environments

Can you update the description to include the refactoring of settings as well? I don't think any CI updates need to be made as we have the run_all_tests.py script encapsulating all the testing, but double check this

@Mayankm96 Mayankm96 merged commit 46594e0 into main Mar 1, 2025
3 of 4 checks passed
@Mayankm96 Mayankm96 deleted the feature/env-test branch March 1, 2025 14:28
jtigue-bdai pushed a commit that referenced this pull request Apr 14, 2025
# Description

This MR adds a script to run training over a subset of environments. I
have mostly checked this for rsl-rl library.

The script adds the commit tag as a run name allowing us to compare
different versions of code nicely.

## Type of change

- New feature (non-breaking change which adds functionality)

## Screenshots

Example tensorboard log:

```bash
./isaaclab.sh -p tools/run_train_envs.py --lib-name rsl_rl
```


![image](https://github.com/user-attachments/assets/b065bef6-b149-41e7-a5b3-5d98df5c766b)

## Checklist

- [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [x] 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
- [x] I have added my name to the `CONTRIBUTORS.md` or my name already
exists there

---------

Signed-off-by: Mayank Mittal <[email protected]>
ToxicNS pushed a commit to ToxicNS/IsaacLab that referenced this pull request Apr 24, 2025
)

# Description

This MR adds a script to run training over a subset of environments. I
have mostly checked this for rsl-rl library.

The script adds the commit tag as a run name allowing us to compare
different versions of code nicely.

## Type of change

- New feature (non-breaking change which adds functionality)

## Screenshots

Example tensorboard log:

```bash
./isaaclab.sh -p tools/run_train_envs.py --lib-name rsl_rl
```


![image](https://github.com/user-attachments/assets/b065bef6-b149-41e7-a5b3-5d98df5c766b)

## Checklist

- [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [x] 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
- [x] I have added my name to the `CONTRIBUTORS.md` or my name already
exists there

---------

Signed-off-by: Mayank Mittal <[email protected]>
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.

5 participants