Skip to content

Dev#155

Open
vikashplus wants to merge 19 commits intomainfrom
dev
Open

Dev#155
vikashplus wants to merge 19 commits intomainfrom
dev

Conversation

@vikashplus
Copy link
Copy Markdown
Owner

No description provided.

Wenxuan-Zhou and others added 19 commits April 20, 2023 00:03
Point cloud utility functions for mujoco [Draft]
… compliance. Envs now overwrite the correct _reset function. Gym compliance is now preserved across the defaulting envs
- gym/gymnasium compatibility doesn't port over when custom reset is provided
- we are now recommending a call to super().reset(**kwargs) to get the appropritae return type ensuring compatibility
np.product -> np.prod for np 2.0 compatibility
1. FEATURE: Explicitely marks the closure of the trace
2. BUGFIX: Introduces create_datum. create_dataset is updated and is now reserved for adding full datasets (shouldn't be edited later). If appending is needed first create_datum() and then follow it with repeated calls to append_datum().
3. BUGFIX: verify_type is stricter now. It goes beyond checking for root dtypes. It thoroughly checks for type, shapes, keys - even for nested structures. This implies that dict{datum} also needs to maintain its internal keys to support append leading to further consistency
4. FEATURE: Much like rendering of rgb datasets, now trace supports plotting of numerical datasets using a list(grp_keys)_ng and list(list(dataset_keys))_ng
5. Adds multiple tests to check for data consistency and plotting consistency
@andreh1111
Copy link
Copy Markdown
Collaborator

andreh1111 commented May 7, 2026

  • there are still a few obs, rwd, done, info = env.forward() / env.step() to be switched to obs, rwd, done, *_, info = env.forward() / env.step()
  • I think env.env should be replaced with env.unwrapped, e.g. in robohive/tutorials/ee_teleop.py
  • github actions are failing

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