Skip to content

[BUG] Diffusion Policy Bug#1083

Open
rrzhang139 wants to merge 2 commits into
mani-skill:mainfrom
rrzhang139:diffusion-policy-bug
Open

[BUG] Diffusion Policy Bug#1083
rrzhang139 wants to merge 2 commits into
mani-skill:mainfrom
rrzhang139:diffusion-policy-bug

Conversation

@rrzhang139

Copy link
Copy Markdown

Shifted the starting index of the action slice.

Ensures that the first predicted action (e.g. a_t) correctly follows the last observation (o_t), matching standard diffusion-policy convention (Chi et al.).

@StoneT2000

Copy link
Copy Markdown
Member

Have you tried running the training baselines yet?

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.

2 participants