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

New package: AStarGridSearch v0.1.0 #120282

Merged

Conversation

JuliaRegistrator
Copy link
Contributor

@JuliaRegistrator JuliaRegistrator commented Nov 27, 2024

Initial release of AStarGridSearch. Path searching and distance calculation on a 2D grid.

UUID: 892abc4-b326-442a-ba8e-5eb8be33ac6f
Repo: https://github.com/ngharrison/AStarGridSearch.jl.git
Tree: f3820ccff8ea9c18909af777d2a73ca21d7235ca

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
@JuliaRegistrator JuliaRegistrator force-pushed the registrator-astargridsearch-892abc44-v0.1.0-4af42526d0 branch from dccbb2f to 86dd90f Compare November 27, 2024 18:02
Copy link
Contributor

Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human.

1. New package registration

Please make sure that you have read the package naming guidelines.

2. AutoMerge Guidelines are all met! ✅

Your new package registration met all of the guidelines for auto-merging and is scheduled to be merged when the mandatory waiting period (3 days) has elapsed.

3. To pause or stop registration

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@gdalle
Copy link
Contributor

gdalle commented Nov 28, 2024

[noblock]
Hi, congrats on the new package! I just wanted to check whether you are aware of GridGraphs, which may implement most of what you need? We're currently revamping it though so we can definitely add / remove stuff.

@JuliaTagBot JuliaTagBot added the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Nov 28, 2024
@ngharrison
Copy link

@gdalle I'm breaking this package out from a bigger project I've been working on. When I first started it over a year and a half ago, I tried out the A* alg in Graphs, but it was too slow for me. I can't remember now if I saw GridGraphs or not, but I turned to making my own implementation anyway due to requirements for simplicity and efficiency. In particular, I needed something that could calculate path distances on a discretized 2D map and serve as part of a cost function in an optimization routine. The cost function is called hundreds or thousands of times for the same starting location and different goal locations, so I use A* and save a cost matrix that can be shared between runs. I can't say what the performance comparison would be with your package now/future.

I've worked to register this as its own package to both improve the experience for collaborators working on my other packages that use it as a dependency as well as to let other people use it on its own if they have a similar need. Since this package has only one main feature, I have imagined it being absorbed into a bigger package system. However, I've envisioned more generally path planning in 2D/3D space rather than purely graph-based applications. For context, my applications deal with robotics.

[noblock]

@gdalle
Copy link
Contributor

gdalle commented Dec 1, 2024

[noblock]
Fair enough. If you can provide an MWE showing how your code is faster than the Graphs.jl equivalent, I'll be interested to see why

@JuliaTagBot JuliaTagBot removed the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Dec 1, 2024
@JuliaTagBot JuliaTagBot merged commit 5bee0aa into master Dec 1, 2024
14 checks passed
@JuliaTagBot JuliaTagBot deleted the registrator-astargridsearch-892abc44-v0.1.0-4af42526d0 branch December 1, 2024 22:34
@ngharrison
Copy link

Thanks. When I get some time, I hope to add some info about the relation to other packages. I can make a comparison then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants