-
Notifications
You must be signed in to change notification settings - Fork 66
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
Added basic GUI controls with stepping, restart, diagonals option. Node info. Refactoring. #9
Conversation
Split out pathfinder and map to separate files. implemented run/pause, step and restart buttons. Moved search state information output to a text draw rather than html insertions.
Fixed bug where the visual heuristic wasn't being run for 4-way movement. Added printing of open node info on mouseover.
# Conflicts: # spot.js
corrected mistake in node info printing.
Ooops, I ended up going in order and merged #4. Can you refactor this? |
Yeah, I'll get onto it later this evening. Early stage changes to this sort of quick and dirty code are always going to end up with breaking merges. |
# Conflicts: # sketch.js # spot.js
Clearing of timings on restart.
Moved removeFromArray function to Pathfinder.
Okay, done. I think the rendering stuff needs pulling out of Spot and put into a MapRenderer or similar but this will do for now before more pull requests pile up. |
This is great, thank you! I think it might be a nice improvement to change the code to use |
Sure. I'd have no major objections to replacing the UI. The reasons I just hacked together some UI code are just:
|
Thanks! Yes, p5.dom.js is a library I teach with that goes well with the main p5.js canvas rendering so it's a great choice for here. Thanks for getting this started! |
I don't know if you'd rather take this easy merge and make #4 more difficult or go in order and bounce this back to resolve conflicts after that one. I'm easy either way.