-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/waypoints file #17
Open
danielbrownmsm
wants to merge
23
commits into
main
Choose a base branch
from
feat/waypoints_file
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ut not the right waypoints yet also you can't do anything with it other than stare at it
gotta be a better way to center the image than guess and check though
not that it will stop me
got it to the point where it looks kinda decent, but it has no functionality and for some reason the buttons aren't acting as buttons
…gs to be constants need to figure out how WaypointHandler is gonna keep track of stuff
…ther now still need to figure out the actual editing/drag/drop funcionality but it's kind of close
…ht waypoint, but it still doesn't clear old waypoints from the screen, and <add> and <modify> don't work/exist
… the main class. 3 hours out of Indianapolis, so there's probably a lot of mistakes made here
… screen in Ohio, few hours out of Detroit
Don't merge this until it's been tested on more recent data, I think it might be flipped 90 degrees or something, and I want to see if I can mess with some graphical stuff |
…ad of left-click so the user can like use the zoom function and stuff
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Have GPS waypoints be in a separate .csv file instead of being hardcoded, allowing us to more easily edit the waypoints, and not have to rebuilt the code every time we want to make a minor change.
Also adds a bare-bones way to edit the waypoints graphically using matplotlib so you can see where the waypoints actuallly are in comparison to the path the robot follows