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

Add A* Pathfinder Extensions #1828

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Conversation

YogaindoCR
Copy link

Find the fastest path with high performance in real-time!

image
Feature:

  • Direct import from list
  • Diagonal Movement
  • Weighted Path (Slower paths)
  • Jump Point Search Algorithm Implemented

A thumbnail is needed for the extension 👀

image
White Square Road, Black Square Walls and Yellow Square Weighted.

@YogaindoCR
Copy link
Author

All done! and... idk how to fix the third one 👀

@Vxerz
Copy link

Vxerz commented Jan 4, 2025

to fix the third one you gotta use Prettier on the file
https://prettier.io/docs/en/install

Copy link
Contributor

@LilyMakesThings LilyMakesThings left a comment

Choose a reason for hiding this comment

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

More comments would go a long way in ensuring this is maintainable.

extensions/YogaindoCR/astarpathfind.js Outdated Show resolved Hide resolved
"use strict";

/* Code Transparancy
* A* Star Pathfinder was created with the help of AI
Copy link
Contributor

Choose a reason for hiding this comment

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

So long as you understand everything your code does, given that the rules regarding LLMs were inexplicably removed a year ago with no reason attributed in the commit log, this should be fine.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah I wonder why they were removed. (the rules against LLM's)

Copy link
Author

Choose a reason for hiding this comment

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

It should be fine :^.
But if this extensions got removed or rejected, It's understandable.

Run prettier & Add username to ID
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.

4 participants