This repository has been archived by the owner on Jun 5, 2019. It is now read-only.
forked from autowarefoundation/autoware
-
Notifications
You must be signed in to change notification settings - Fork 3
As/111 live traffic detector #13
Merged
JWhitleyWork
merged 15 commits into
as/111_live_traffic
from
as/111_live_traffic_detector
May 22, 2019
Merged
As/111 live traffic detector #13
JWhitleyWork
merged 15 commits into
as/111_live_traffic
from
as/111_live_traffic_detector
May 22, 2019
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
as-snehagn
approved these changes
May 20, 2019
31a9570
to
fede3e8
Compare
4bc6e01
to
2772cff
Compare
The rebase + the fix I applied to |
@as-snehagn / @shan-as I modified |
JWhitleyWork
commented
May 22, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments.
...rception/detection/trafficlight_recognizer/launch/traffic_light_recognizer_tensorflow.launch
Outdated
Show resolved
Hide resolved
...rception/detection/trafficlight_recognizer/launch/traffic_light_recognizer_tensorflow.launch
Outdated
Show resolved
Hide resolved
cbradyas
approved these changes
May 22, 2019
shan-as
approved these changes
May 22, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested & Verified in Peoria
JWhitleyWork
pushed a commit
that referenced
this pull request
Jun 4, 2019
* Add plugin edit tool and update plugins * Update the code for compatibility with python3 * Skip test of unbuilt packages * Two other minor rebase fixes. * First pass at adding region_tlr_tensorflow and roi_pub. * Adding 1.13 to FindTensorFlow.cmake. * Removing FindCuDNN.cmake. Changing FindTensorFlow variables. * Adding proper Python folder structure. * Adding new nodes to interface.yaml. * Minor formatting fix. * Making encoding in roi_pub type-specific. * Fixing roi_pub for images with rggb8 format. * Launch file for region_tlr_tensorflow * Renaming launch file for consistency. * Removing unused portion of launch file.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Adds
region_tlr_tensorflow
(Joe's traffic light recognizer) androi_pub
totrafficlight_recognizer
. It builds and installs correctly on a machine with CUDA and TensorFlow (tried checking for CuDNN but it can't be done properly with CMake anymore) and skips installation ofregion_tlr_tensorflow
if either of these are not found. This is likely not the final version but it is required for testing on Monday (5/20/19).