Path tracer improvements #565
Merged
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
…se in events.yml Signed-off-by: Nate Koenig <[email protected]>
acschang
reviewed
Sep 16, 2020
iche033
approved these changes
Sep 16, 2020
nkoenig
added a commit
that referenced
this pull request
Dec 17, 2020
* bump to citadel, update comms visualization * Adding breadcrumbs to x1 config 7 & 8 * Added additional breadcrumbs * Update to citadel * Prevent breadcrumb topics * Added breadcrumb handling to more launch files * Apply patch * Adjust spawn location of the breadcrumbs * Update cave_circuit.ign to call the corect spawner method (#409) Signed-off-by: Addisu Z. Taddese <[email protected]> * Add disable_physics_time param to breadcrumb plugin to auto disable them Signed-off-by: Ian Chen <[email protected]> * Adding team base * One team base only * Cleanup after move from bitbucket * Cleanup * Remove extra whitespace * Use https for git * Cave Qual release * Update Citadel and Urban Circuit * Update Citadel and Urban Circuit * bump to citadel, update comms visualization * Update to citadel * bump to citadel, update comms visualization * Cleanup after rebase * Updates based on review * Missed one pose publisher * one more RawPose Signed-off-by: Ian Chen <[email protected]> * Minor cmake tweaks Signed-off-by: Nate Koenig <[email protected]> * Change colors for comms visualization. Signed-off-by: Nate Koenig <[email protected]> * Fix docs Signed-off-by: Nate Koenig <[email protected]> * Azeey/marsupial (#424) * Add marsupial robots for cave_circuit * Update example command * Apply spawnWorldYaw to marsupial offsets * Add marsupial vehicle support to cloudsim Signed-off-by: Addisu Z. Taddese <[email protected]> * Update cloudsim_bridge and json2docker Signed-off-by: Addisu Z. Taddese <[email protected]> * Suppress DetachableJoint missing child warning Signed-off-by: Addisu Z. Taddese <[email protected]> * Added platform Signed-off-by: Nate Koenig <[email protected]> * Fix platform position Signed-off-by: Nate Koenig <[email protected]> * Combined spawn Signed-off-by: Nate Koenig <[email protected]> * Update submitted models Signed-off-by: Nate Koenig <[email protected]> * Merge cave_circuit.ign to urban_circuit.ign, tunnel_circuit_practice.ign, and competition.ign Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Addisu Z. Taddese <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Add missing changes from PRs #402 and #403 Signed-off-by: Addisu Z. Taddese <[email protected]> * Adjust platform height Signed-off-by: Nate Koenig <[email protected]> * Fix deprecation Signed-off-by: Nate Koenig <[email protected]> * Fix build Signed-off-by: Nate Koenig <[email protected]> * Update to visibility RF model (#440) * Testing. * Remove debug code. * Update to visibility RF model to account for radio characteristics and message size. * Updated the static number of bytes in a message for communication visualization from 5 to 100. Co-authored-by: Carlos Aguero <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * citadel dockerfiles Signed-off-by: Nate Koenig <[email protected]> * Merged with master Signed-off-by: Nate Koenig <[email protected]> * Fix dependencies Signed-off-by: Nate Koenig <[email protected]> * Download models Signed-off-by: Nate Koenig <[email protected]> * Path tracer (#545) * Added a path tracer program that visualizes robot paths, artifacts, and artifact reports. Signed-off-by: Nate Koenig <[email protected]> * Added path tracer files Signed-off-by: Nate Koenig <[email protected]> * Cleanup build and address comments Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Break out model download into separate docker image. Signed-off-by: Nate Koenig <[email protected]> * Cleanup Signed-off-by: Nate Koenig <[email protected]> * Update docker image name Signed-off-by: Nate Koenig <[email protected]> * Updating docker Signed-off-by: Nate Koenig <[email protected]> * Fixed merge Signed-off-by: Nate Koenig <[email protected]> * Handle log files with no events.yml file, and also handle an error case in events.yml (#565) Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Path tracer rtf (#596) * Modify path tracer to use an RTF value Signed-off-by: Nate Koenig <[email protected]> * Added more output Signed-off-by: Nate Koenig <[email protected]> * Account for time required to process the step Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Path tracer colors (#599) * Modify path tracer to use an RTF value Signed-off-by: Nate Koenig <[email protected]> * Start of path tracer color yaml configuration Signed-off-by: Nate Koenig <[email protected]> * Support yaml configuration for colors and rtf Signed-off-by: Nate Koenig <[email protected]> * Update to use DRY suggestion Signed-off-by: Nate Koenig <[email protected]> * Update subt_ign/src/path_tracer.cc Co-authored-by: Michael Carroll <[email protected]> Co-authored-by: Nate Koenig <[email protected]> Co-authored-by: Michael Carroll <[email protected]> * Merge from master (#621) Signed-off-by: Carlos Agüero <[email protected]> * Missing include Signed-off-by: Nate Koenig <[email protected]> * Merged with master Signed-off-by: Nate Koenig <[email protected]> * Disable truth_controller * Update to use Ignition Dome Signed-off-by: Nate Koenig <[email protected]> * Updating dockerfiles Signed-off-by: Nate Koenig <[email protected]> * Updating subt_sim_entry dockerfile Signed-off-by: Nate Koenig <[email protected]> * Updating model download Signed-off-by: Nate Koenig <[email protected]> * Update dockerfile Signed-off-by: Nate Koenig <[email protected]> * Update dockefile Signed-off-by: Nate Koenig <[email protected]> * Update subt_sim_entry Signed-off-by: Nate Koenig <[email protected]> * Update docker Signed-off-by: Nate Koenig <[email protected]> * Fixing docker Signed-off-by: Nate Koenig <[email protected]> * Improve git clone Signed-off-by: Nate Koenig <[email protected]> * Add back in ros_ign bridge to cloudsim_sim dockerfile Signed-off-by: Nate Koenig <[email protected]> * Update all urban .dat files. (#721) Signed-off-by: Carlos Agüero <[email protected]> * Update all tunnel .dat files. (#723) Signed-off-by: Carlos Agüero <[email protected]> * Update dockerfiles Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Ian Chen <[email protected]> Co-authored-by: Nate Koenig <[email protected]> Co-authored-by: Addisu Taddese <[email protected]> Co-authored-by: acschang <[email protected]> Co-authored-by: Carlos Aguero <[email protected]> Co-authored-by: Michael Carroll <[email protected]> Co-authored-by: Carlos Agüero <[email protected]>
nkoenig
added a commit
that referenced
this pull request
Jan 5, 2021
* bump to citadel, update comms visualization * Adding breadcrumbs to x1 config 7 & 8 * Added additional breadcrumbs * Update to citadel * Prevent breadcrumb topics * Added breadcrumb handling to more launch files * Apply patch * Adjust spawn location of the breadcrumbs * Update cave_circuit.ign to call the corect spawner method (#409) Signed-off-by: Addisu Z. Taddese <[email protected]> * Add disable_physics_time param to breadcrumb plugin to auto disable them Signed-off-by: Ian Chen <[email protected]> * Adding team base * One team base only * Cleanup after move from bitbucket * Cleanup * Remove extra whitespace * Use https for git * Cave Qual release * Update Citadel and Urban Circuit * Update Citadel and Urban Circuit * bump to citadel, update comms visualization * Update to citadel * bump to citadel, update comms visualization * Cleanup after rebase * Updates based on review * Missed one pose publisher * one more RawPose Signed-off-by: Ian Chen <[email protected]> * Minor cmake tweaks Signed-off-by: Nate Koenig <[email protected]> * Change colors for comms visualization. Signed-off-by: Nate Koenig <[email protected]> * Fix docs Signed-off-by: Nate Koenig <[email protected]> * Azeey/marsupial (#424) * Add marsupial robots for cave_circuit * Update example command * Apply spawnWorldYaw to marsupial offsets * Add marsupial vehicle support to cloudsim Signed-off-by: Addisu Z. Taddese <[email protected]> * Update cloudsim_bridge and json2docker Signed-off-by: Addisu Z. Taddese <[email protected]> * Suppress DetachableJoint missing child warning Signed-off-by: Addisu Z. Taddese <[email protected]> * Added platform Signed-off-by: Nate Koenig <[email protected]> * Fix platform position Signed-off-by: Nate Koenig <[email protected]> * Combined spawn Signed-off-by: Nate Koenig <[email protected]> * Update submitted models Signed-off-by: Nate Koenig <[email protected]> * Merge cave_circuit.ign to urban_circuit.ign, tunnel_circuit_practice.ign, and competition.ign Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Addisu Z. Taddese <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Add missing changes from PRs #402 and #403 Signed-off-by: Addisu Z. Taddese <[email protected]> * Adjust platform height Signed-off-by: Nate Koenig <[email protected]> * Fix deprecation Signed-off-by: Nate Koenig <[email protected]> * Fix build Signed-off-by: Nate Koenig <[email protected]> * Update to visibility RF model (#440) * Testing. * Remove debug code. * Update to visibility RF model to account for radio characteristics and message size. * Updated the static number of bytes in a message for communication visualization from 5 to 100. Co-authored-by: Carlos Aguero <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * citadel dockerfiles Signed-off-by: Nate Koenig <[email protected]> * Merged with master Signed-off-by: Nate Koenig <[email protected]> * Fix dependencies Signed-off-by: Nate Koenig <[email protected]> * Download models Signed-off-by: Nate Koenig <[email protected]> * Path tracer (#545) * Added a path tracer program that visualizes robot paths, artifacts, and artifact reports. Signed-off-by: Nate Koenig <[email protected]> * Added path tracer files Signed-off-by: Nate Koenig <[email protected]> * Cleanup build and address comments Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Break out model download into separate docker image. Signed-off-by: Nate Koenig <[email protected]> * Cleanup Signed-off-by: Nate Koenig <[email protected]> * Update docker image name Signed-off-by: Nate Koenig <[email protected]> * Updating docker Signed-off-by: Nate Koenig <[email protected]> * Fixed merge Signed-off-by: Nate Koenig <[email protected]> * Handle log files with no events.yml file, and also handle an error case in events.yml (#565) Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Path tracer rtf (#596) * Modify path tracer to use an RTF value Signed-off-by: Nate Koenig <[email protected]> * Added more output Signed-off-by: Nate Koenig <[email protected]> * Account for time required to process the step Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Nate Koenig <[email protected]> * Path tracer colors (#599) * Modify path tracer to use an RTF value Signed-off-by: Nate Koenig <[email protected]> * Start of path tracer color yaml configuration Signed-off-by: Nate Koenig <[email protected]> * Support yaml configuration for colors and rtf Signed-off-by: Nate Koenig <[email protected]> * Update to use DRY suggestion Signed-off-by: Nate Koenig <[email protected]> * Update subt_ign/src/path_tracer.cc Co-authored-by: Michael Carroll <[email protected]> Co-authored-by: Nate Koenig <[email protected]> Co-authored-by: Michael Carroll <[email protected]> * Merge from master (#621) Signed-off-by: Carlos Agüero <[email protected]> * Missing include Signed-off-by: Nate Koenig <[email protected]> * Merged with master Signed-off-by: Nate Koenig <[email protected]> * Disable truth_controller * Update to use Ignition Dome Signed-off-by: Nate Koenig <[email protected]> * Updating dockerfiles Signed-off-by: Nate Koenig <[email protected]> * Updating subt_sim_entry dockerfile Signed-off-by: Nate Koenig <[email protected]> * Updating model download Signed-off-by: Nate Koenig <[email protected]> * Update dockerfile Signed-off-by: Nate Koenig <[email protected]> * Update dockefile Signed-off-by: Nate Koenig <[email protected]> * Update subt_sim_entry Signed-off-by: Nate Koenig <[email protected]> * Update docker Signed-off-by: Nate Koenig <[email protected]> * Fixing docker Signed-off-by: Nate Koenig <[email protected]> * Improve git clone Signed-off-by: Nate Koenig <[email protected]> * Add back in ros_ign bridge to cloudsim_sim dockerfile Signed-off-by: Nate Koenig <[email protected]> * Tunnel connection points Signed-off-by: Nate Koenig <[email protected]> * added subt_tunnel_staging_area Signed-off-by: Nate Koenig <[email protected]> * Fixed connection points Signed-off-by: Nate Koenig <[email protected]> Co-authored-by: Ian Chen <[email protected]> Co-authored-by: Nate Koenig <[email protected]> Co-authored-by: Addisu Taddese <[email protected]> Co-authored-by: acschang <[email protected]> Co-authored-by: Carlos Aguero <[email protected]> Co-authored-by: Michael Carroll <[email protected]> Co-authored-by: Carlos Agüero <[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.
Support older log files, which may not have an
events.yml.Also fix a bug in the events.yml file which looks like:
Note the missing colons after
elapsed_real_timeandelapsed_sim_time.Signed-off-by: Nate Koenig [email protected]