Skip to content

Conversation

@Toni-SM
Copy link
Contributor

@Toni-SM Toni-SM commented Mar 7, 2025

Description

This PR adds the implementation to generate Isaac Lab's external project or internal task.

Just run ./isaaclab.sh --new or isaaclab.bat --new (or -n) and let yourself go!

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

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
Contributor

@Mayankm96 Mayankm96 left a comment

Choose a reason for hiding this comment

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

Amazing work! Thank you for pulling this together!

Quick feedback:

  1. Can we move the template directory to tools to have all these utilities together?
  2. Need to also modify instructions here: https://isaac-sim.github.io/IsaacLab/main/source/overview/developer-guide/template.html
  3. If we modify something in core, we need to modify the env and env_cfg inside the tasks directory in the template?

@Toni-SM
Copy link
Contributor Author

Toni-SM commented Mar 7, 2025

Quick feedback:

  1. Can we move the template directory to tools to have all these utilities together?
  2. Need to also modify instructions here: https://isaac-sim.github.io/IsaacLab/main/source/overview/developer-guide/template.html
  3. If we modify something in core, we need to modify the env and env_cfg inside the tasks directory in the template?
  1. Done
  2. Done
  3. Yes, it is. We could copy (and replace with the new names) the tasks directly from the Isaac Lab code (in this case for the Cartpole/Cart-Double-Pendulum). However, I have chosen to separate the task files in order to add (in the near future), comments and links to help users to re-do the task according to their needs. Such comments and links would clutter the current task files in isaaclab_tasks if we were to add them there.

### Building Isaac Lab Base Image
Currently, we don't have the Docker for Isaac Lab publicly available. Hence, you'd need to build the docker image
Copy link
Contributor

Choose a reason for hiding this comment

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

we do have the prebuilt Isaac Lab docker now

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Inherited from https://github.com/isaac-sim/IsaacLabExtensionTemplate.
Should we remove those steps?

### Building Isaac Lab Template Image
Following above, you can build the docker container for this project. It is called `isaac-lab-template`. However,
Copy link
Contributor

Choose a reason for hiding this comment

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

do these docker steps still apply for the new template that's generated?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Inherited from https://github.com/isaac-sim/IsaacLabExtensionTemplate.
Should we remove those steps?

Copy link
Contributor

Choose a reason for hiding this comment

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

would still be good to support docker. do you mind verifying the steps to see we can still follow them to use the new template in docker?

Toni-SM and others added 8 commits March 10, 2025 10:11
Co-authored-by: Kelly Guo <kellyg@nvidia.com>
Signed-off-by: Toni-SM <toni.semu@gmail.com>
Co-authored-by: Kelly Guo <kellyg@nvidia.com>
Signed-off-by: Toni-SM <toni.semu@gmail.com>
Co-authored-by: Kelly Guo <kellyg@nvidia.com>
Signed-off-by: Toni-SM <toni.semu@gmail.com>
Co-authored-by: Kelly Guo <kellyg@nvidia.com>
Signed-off-by: Toni-SM <toni.semu@gmail.com>
@kellyguo11 kellyguo11 changed the title Isaac Lab's external project / internal task template generator Adds new external project / internal task template generator Mar 11, 2025
Signed-off-by: Kelly Guo <kellyguo123@hotmail.com>
@kellyguo11 kellyguo11 merged commit 80e3964 into main Mar 13, 2025
4 of 5 checks passed
@kellyguo11 kellyguo11 deleted the feature/template_generator branch March 13, 2025 03:47
jtigue-bdai pushed a commit that referenced this pull request Apr 14, 2025
# Description

This PR adds the implementation to generate Isaac Lab's external project
or internal task.

Just run `./isaaclab.sh --new` or `isaaclab.bat --new` (or `-n`) and let
yourself go!

## Type of change

- New feature (non-breaking change which adds functionality)
- This change requires a documentation update

## 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
- [ ] 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

<!--
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
-->

---------

Signed-off-by: Toni-SM <toni.semu@gmail.com>
Signed-off-by: Kelly Guo <kellyguo123@hotmail.com>
Co-authored-by: Kelly Guo <kellyg@nvidia.com>
ToxicNS pushed a commit to ToxicNS/IsaacLab that referenced this pull request Apr 24, 2025
…im#2039)

# Description

This PR adds the implementation to generate Isaac Lab's external project
or internal task.

Just run `./isaaclab.sh --new` or `isaaclab.bat --new` (or `-n`) and let
yourself go!

## Type of change

- New feature (non-breaking change which adds functionality)
- This change requires a documentation update

## 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
- [ ] 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

<!--
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
-->

---------

Signed-off-by: Toni-SM <toni.semu@gmail.com>
Signed-off-by: Kelly Guo <kellyguo123@hotmail.com>
Co-authored-by: Kelly Guo <kellyg@nvidia.com>
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.

4 participants