Skip to content
This repository has been archived by the owner on Sep 15, 2022. It is now read-only.

Update server.py #8

Merged
merged 3 commits into from
Sep 1, 2017
Merged

Update server.py #8

merged 3 commits into from
Sep 1, 2017

Conversation

awbrown90
Copy link
Contributor

adding dbw_enable status change from the simulator's manual control button

adding dbw_enable status change from the simulator's manual control button

@sio.on('connect')
def connect(sid, environ):
print("connect ", sid)
bridge.publish_dbw_status(True)
bridge.publish_dbw_status(dbw_enable)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this line

@@ -29,6 +30,10 @@ def send(topic, data):

@sio.on('telemetry')
def telemetry(sid, data):
global dbw_enable
if(data["dbw_enable"] != dbw_enable):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the parantheses

changed to hard coded False when starting, its False because car starts in manual mode and dbw_enable is false at that state. Removed the parenthesis
remove the unneeded line
@awbrown90 awbrown90 merged commit 00f57c5 into master Sep 1, 2017
@awbrown90 awbrown90 deleted the awbrown90-dbw_patch-1 branch September 1, 2017 23:16
lake4790k pushed a commit to lake4790k/CarND-Capstone that referenced this pull request Jan 17, 2018
WIP: tl_detector fix missing position msg bug
samipshah pushed a commit to samipshah/CarND-Capstone that referenced this pull request Jan 22, 2018
rajatsharma01 referenced this pull request in rajatsharma01/CarND-Capstone Oct 9, 2018
dylanbrandtner pushed a commit to dylanbrandtner/CarND-Capstone that referenced this pull request Apr 5, 2019
Verichev pushed a commit to Verichev/CarND-Capstone that referenced this pull request Jul 16, 2019
Issue # 7: Create a shared_utils package for use by other packages
bwosh pushed a commit to bwosh/CarND-Capstone that referenced this pull request Mar 19, 2020
Update Blazej's email to udacity email
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants