-
Notifications
You must be signed in to change notification settings - Fork 5
Add six new environments: BlocksRanking, StackCups, StackBlocksTwo, MatchObjectContainer, and PlaceObjectDrawer #70
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: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Lihao <lihao@dhcp-10-105-153-63.wireless.northwestern.private>
Co-authored-by: yuanhaonan <yuanhaonan@dexforce.top>
…exforceW1 config construction (#10) Co-authored-by: Jietao Chen <chenjietao@dexforce.com>
Co-authored-by: chenjian <chenjian@dexforce.com>
Co-authored-by: zengkehuan <zengkehuan@dexforece.com> Co-authored-by: yuecideng <dengyueci@qq.com>
Co-authored-by: zengkehuan <zengkehuan@dexforece.com>
…nings, optimize loops (#19) Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: yuecideng <dengyueci@qq.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: yuecideng <dengyueci@qq.com>
Co-authored-by: yuanhaonan <yuanhaonan@dexforce.top>
Co-authored-by: chenjian <chenjian@dexforce.com>
#27) Co-authored-by: chenjian <chenjian@dexforce.com> Co-authored-by: Yueci Deng <dengyueci@qq.com>
Co-authored-by: Jietao Chen <chenjietao@dexforce.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: yuecideng <dengyueci@qq.com>
Co-authored-by: yuanhaonan <yuanhaonan@dexforce.top> Co-authored-by: yuecideng <dengyueci@qq.com>
Co-authored-by: yuanhaonan <yuanhaonan@dexforce.top> Co-authored-by: Yueci Deng <dengyueci@qq.com>
Co-authored-by: Jietao Chen <chenjietao@dexforce.com>
Co-authored-by: chenjian <chenjian@dexforce.com>
Co-authored-by: yuanhaonan <yuanhaonan@dexforce.top>
Co-authored-by: guilong li <liguilong46@gmail.com>
| }, | ||
| "sensor": [ | ||
| { | ||
| "sensor_type": "StereoCamera", |
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.
We may follow the common setup of cobotmagic robot (3 camera with the similar params)
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.
I'v update sensors.
| ) | ||
|
|
||
|
|
||
| def randomize_rigid_object_body_scale( |
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.
Could we remove these two functors if they will not be used anymore
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.
I think it is better to keep it. Maybe some environments in the future may use them.
| env: EmbodiedEnv, | ||
| env_ids: Union[torch.Tensor, None], | ||
| entity_cfg: SceneEntityCfg, | ||
| base_color: list[float] | None = None, |
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.
Could we just use VisualMaterialCfg? It can also created from dict
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.
Yes. I'v use VisualMaterialCfg in set_rigid_object_visual_material:
cfg = VisualMaterialCfg(
uid=mat_uid,
base_color=base_color if base_color is not None else [0.5, 0.5, 0.5, 1.0],
metallic=metallic,
roughness=roughness,
)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.
I mean the args related to material are replaced by just VisualMaterialCfg. For example:
def set_rigid_object_visual_material(
env: EmbodiedEnv,
env_ids: Union[torch.Tensor, None],
entity_cfg: SceneEntityCfg,
mat_cfg: VisualMaterialCfg,Since we may extend and change the param in VisualMaterialCfg. And we do not want to change the functor args when changes have happened to VisualMaterialCfg
|
|
||
|
|
||
| @register_env("BlocksRankingRGB-v1", max_episode_steps=600) | ||
| class BlocksRankingRGBEnv(EmbodiedEnv): |
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.
For these six task, we may create a simple expert trajectory for demonstration
Co-authored-by: chenjian <chenjian@dexforce.com> Co-authored-by: yuecideng <dengyueci@qq.com>
Description
This PR adds six new environments to
EmbodiChain.1. BlocksRankingRGB-v1 (
BlocksRankingRGBEnv)2. BlocksRankingSize-v1 (
BlocksRankingSizeEnv)3. StackCups-v1 (
StackCupsEnv)4. StackBlocksTwo-v1 (
StackBlocksTwoEnv)5. MatchObjectContainer-v1 (
MatchObjectContainerEnv)container_cube.container_sphere.6. PlaceObjectDrawer-v1 (
PlaceObjectDrawerEnv)inner_boxarea.Type of change
Screenshots
Checklist
black .command to format the code base.