Skip to content
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

Round position in add_cost to reduce inaccuracy #100

Merged
merged 1 commit into from
Dec 13, 2020
Merged

Round position in add_cost to reduce inaccuracy #100

merged 1 commit into from
Dec 13, 2020

Conversation

shostyn
Copy link
Contributor

@shostyn shostyn commented Dec 13, 2020

No description provided.

@eladyaniv01 eladyaniv01 merged commit 6411802 into eladyaniv01:develop Dec 13, 2020
eladyaniv01 added a commit that referenced this pull request Dec 13, 2020
* Round position in add_cost to reduce inaccuracy (#100)

Co-authored-by: shostyn <74581079+shostyn@users.noreply.github.com>
@shostyn shostyn deleted the develop branch December 14, 2020 02:07
eladyaniv01 added a commit that referenced this pull request Dec 20, 2020
* init commit develop

* feat: MapData now accepts a `corner_distance` variable that will determin the corner distance calculation

* Round position in add_cost to reduce inaccuracy (#100)

* removeme

* refactor: left right of choke are now side_a and side_b,   sides are computed accuratly now

* feat: Debugger now plots choke side a and side b with text indicators (sA, sB)

* feat: Polygon now has top,bottom,right,left properties

* test: testing the choke sides are included in choke points

* setup bump 0.0.77

* chore(release): 0.0.77

* docs(release): pretty up changelog

* docs(release): pretty up changelog

* fix: temp fix for climber grid test

* Update CHANGELOG.md

* Update CHANGELOG.md

* feat: use custom implementation ofr pyastar

downgrade numpy to 1.19.3  for windows users

* feat: fix pyastar ref in requirements

* Changed point and weight inputs from int to float

Rewrote add_cost function:
 - Changed open disk to closed disk (add 0.001~)
 - Removed bounds safety check (disk should already do this)
 - Changed order of safe/np.any call to improve performance

Changed all input from ints to floats for better accuracy

* Added 0.01 to find lowest cost points to match add cost

Co-authored-by: shostyn <74581079+shostyn@users.noreply.github.com>
Co-authored-by: shostyn <sashahostyn@yahoo.ca>
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