Skip to content

InflationLayer: Better Radius Parameter Limits + Standard Decay Function#81

Merged
amock merged 3 commits intonaturerobots:mainfrom
amock:better-inflation-ranges
Jul 7, 2025
Merged

InflationLayer: Better Radius Parameter Limits + Standard Decay Function#81
amock merged 3 commits intonaturerobots:mainfrom
amock:better-inflation-ranges

Conversation

@amock
Copy link
Collaborator

@amock amock commented Jul 4, 2025

I made some changes to the inflation layer. I

  1. increased the range limits for radius parameters
  2. replaced the cosine cost decay function with the standard costmap2d function for better comparability
  3. removed unused functions

@amock amock changed the title Better Inflation Parameter Ranges + Standard Inflation Decay Function InflationLayer: Better Radius Parameter Limits + Standard Decay Function Jul 4, 2025
Copy link
Contributor

@Cakem1x Cakem1x left a comment

Choose a reason for hiding this comment

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

lgtm

@amock
Copy link
Collaborator Author

amock commented Jul 4, 2025

Why is "build_and_test" pending?

@Cakem1x
Copy link
Contributor

Cakem1x commented Jul 7, 2025

Why is "build_and_test" pending?

Fixed it.

This was caused by the switch to having both humble & jazzy CI. The repo was configured to wait for a check with a slightly different name.

@amock amock merged commit ae5be64 into naturerobots:main Jul 7, 2025
2 checks passed
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