Skip to content

Henry/self cost tuned implementation - #1

Open
henryhshu wants to merge 4 commits into
nathan/obstacle_costfrom
henry/self_cost
Open

henryhshu wants to merge 4 commits into
nathan/obstacle_costfrom
henry/self_cost

Conversation

@henryhshu

Copy link
Copy Markdown
Collaborator

Implemented a supplementary constraint during sampling that penalizes points for being too close to each other:

  • prevents lolipopping and looping of the generated path when an obstacle is hit
  • current implementation combines with virtual_tail_cost to avoid obstacles
  • also added some ui improvements so we can have some usable visualizations for the report

This is a good improvement over the base cost gradient, but we still need to figure out how to incorporate guided diffusion (user input steering) since right now it's just diffusing a path out of nowhere and we want it to actually try to go towards a specified target.

@ingriddleee ingriddleee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some changes so that further down the road we don't run into annoying bugs.

Comment thread itps/common/policies/diffusion/modeling_diffusion.py Outdated
Comment thread itps/test_denoising_viz.py
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