forked from camillasterud/gym-auv
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Respect sensor velocity optimal param #7
Open
krisbrud
wants to merge
52
commits into
EivMeyer:master
Choose a base branch
from
krisbrud:respect-sensor-velocity-optimal-param
base: master
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
Respect sensor velocity optimal param #7
krisbrud
wants to merge
52
commits into
EivMeyer:master
from
krisbrud:respect-sensor-velocity-optimal-param
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
1 make code runable
Ran black formatter on codebase
Refactor train deps
No render default
Config refactor
Observation space refactor
Fix render2d
Simplified moving obstacles environment
Render2d blue background
* Removed render_tiles * Refactored render indicators * Moved text initialization code to constructor * Refactored path and vessel renderers * Gave standard values to viewer width and height * Made RenderableState to make interface for rendering * Added points property to path * Mostly factored out environment of renderer code * Render env doesn't need access to env anymore * WIP removing raw OpenGL code from renderer * WIP before rewriting renderer to pygame * WIP Rendering - before removing more unused functionality * WIP: Made new design, refactoring * Before removing dead renderer code * Before removing make-functions in renderer * Updated environment's use of renderer * Renderer runs, but screen is black * Transform somewhat broken, otherwise functional * Changed sensors to use body coordinates * Renderer now works for TestScenario1, but has x and y swapped relative to NED * Renderer now works with correct NED axis alignment * Removed unused geometry drawing code * Cleaned up render2d * Restructured rendering
* Uncommented colreg env from available envs because unit test fails * Fixed renderer import * Changed back testscenario 1 path * Before updating ubuntu, possibly deleting data * Bumped pygame to 2.1.2 Co-authored-by: Kristian Brudeli <[email protected]>
* Added velocity option to config * Add configurable lidar channels to environment.py * Made n_sensors property not be cached * WIP: Don't return velocities if config says so * Split up vessel code * Remove unused import * Changed enclosing_circle to use shapely point for center * Added utility functions for hierarchical collision detector * Small cleanup in hierarchical utility functions * Add parts of hierarchical collision detector * Working on hierarchical collision detector * Removed if statement used for debugging * Fixed bugs when calculating enclosing circle of vessel obstacle Co-authored-by: Kristian Brudeli <[email protected]>
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.
No description provided.