Sitelet https://github.com/DexterInd/GoPiGo/pull/298
Skip to content

make IR Receiver in easygopigo Python3 ready - #298

Merged
CleoQc merged 1 commit into
DexterInd:developfrom
CleoQc:fix/remote_control
Apr 16, 2018
Merged

CleoQc merged 1 commit into
DexterInd:developfrom
CleoQc:fix/remote_control

Conversation

@CleoQc

@CleoQc CleoQc commented Apr 16, 2018

Copy link
Copy Markdown
Member

On Python3, what we get back from the IR Server is bytes, not strings.

@CleoQc CleoQc self-assigned this Apr 16, 2018
@CleoQc
CleoQc requested a review from RobertLucian April 16, 2018 13:05
@CleoQc
CleoQc merged commit 3882d7c into DexterInd:develop Apr 16, 2018
@CleoQc
CleoQc deleted the fix/remote_control branch April 16, 2018 16:08
RobertLucian added a commit that referenced this pull request Apr 26, 2018
* Remove i2c_mutex, it is now residing in script_tools (#287)

* Feature/easygopigo refactoring (#288)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Add init_<sensor> methods to the easygopigo class so the two robots are kept in sync

* add vscode to  .gitignore

* Feature/systemwide mutex (#290)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Bug/line follower + overall mutex (#291)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Fix import line to its more generic form

* Fix/overallmutexcheck (#292)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Query overallmutex flag every single time

* fix - remove arduino support (#293)

* fix - IR receiver on Stretch (#295)

fix - IR receiver on Stretch

* make IR Receiver in easygopigo Python3 ready (#298)
CleoQc added a commit that referenced this pull request Apr 26, 2018
* Fix Mutex update (#303)

* flip develop to master branch for DexterOS (#304)

* Remove i2c_mutex, it is now residing in script_tools (#287)

* Feature/easygopigo refactoring (#288)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Add init_<sensor> methods to the easygopigo class so the two robots are kept in sync

* add vscode to  .gitignore

* Feature/systemwide mutex (#290)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Bug/line follower + overall mutex (#291)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Fix import line to its more generic form

* Fix/overallmutexcheck (#292)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Query overallmutex flag every single time

* fix - remove arduino support (#293)

* fix - IR receiver on Stretch (#295)

fix - IR receiver on Stretch

* make IR Receiver in easygopigo Python3 ready (#298)
RobertLucian added a commit that referenced this pull request Jun 1, 2018
* Remove i2c_mutex, it is now residing in script_tools (#287)

* Feature/easygopigo refactoring (#288)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Add init_<sensor> methods to the easygopigo class so the two robots are kept in sync

* add vscode to  .gitignore

* Feature/systemwide mutex (#290)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Bug/line follower + overall mutex (#291)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Fix import line to its more generic form

* Fix/overallmutexcheck (#292)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Query overallmutex flag every single time

* fix - remove arduino support (#293)

* fix - IR receiver on Stretch (#295)

fix - IR receiver on Stretch

* make IR Receiver in easygopigo Python3 ready (#298)

* Sync master into develop (#305)

* Fix Mutex update (#303)

* flip develop to master branch for DexterOS (#304)

* Remove i2c_mutex, it is now residing in script_tools (#287)

* Feature/easygopigo refactoring (#288)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Add init_<sensor> methods to the easygopigo class so the two robots are kept in sync

* add vscode to  .gitignore

* Feature/systemwide mutex (#290)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Bug/line follower + overall mutex (#291)

* read_position_str() is also needed here to keep in sync with easygopigo3

* support systemwide mutex

* Fix import line to its more generic form

* Fix/overallmutexcheck (#292)

* read_position_str() is also needed here to keep in sync with easygopigo3

* Query overallmutex flag every single time

* fix - remove arduino support (#293)

* fix - IR receiver on Stretch (#295)

fix - IR receiver on Stretch

* make IR Receiver in easygopigo Python3 ready (#298)

* feature - take line follower out (#306)

* feature - take the line follower install functions out

* feature - remove line follower files

* feature - make modifications to accomodate di_sensors instead

* feature - change to di_sensors paths

* feature - change path due to the presence of a new setup.py

* feature - change paths to DI_Sensors

* feature - add in a function and use regular imports for line_follower

* feature - change the way the install script installs the robot (#308)

* feature - add in bulk changes to the installer scripts

* feature - redefining variables and procedures in install script

* feature - add function for installing a python package and other stuff

* feature - changing just a comment

* feature - change when to exit based on what executable is present

* feature - revert a check for git presence

* feature - exit installer if script_tools fails

* feature - go without using pip for installing large packages

* feature - fix errors during installation phase

* feature - uninstall python packages directly without pip

* feature - fix syntax error in bash for if statement

* feature - skipping the installation of Adafruit_DHT - too much time

* feature - add avrdude & make temp file hidden

* feature - make sure the Dexter folder can be created & add REAMDE

* feature - make the same changes as on the grovepi

* feature - make the same changes to install.sh as on the GrovePi

* feature - fix lingering syntax issues

* feature - correct bad path to setup folder on gopigo

* feature - remove space echoes

* feature - yet again remove space echoes in install script

* feature - add source for 2 functions for avr installer

* feature - replacing "aborting" with "exiting"

* feature - check if script is run with pi user

* feature - correct name of installer

* feature - fix global variable issue

* feature - change how the installation script goes

* feature - slight grammatical change in readme

* feature - change sources for install script (#309)

* feature - add in bulk changes to the installer scripts

* feature - redefining variables and procedures in install script

* feature - add function for installing a python package and other stuff

* feature - changing just a comment

* feature - change when to exit based on what executable is present

* feature - revert a check for git presence

* feature - exit installer if script_tools fails

* feature - go without using pip for installing large packages

* feature - fix errors during installation phase

* feature - uninstall python packages directly without pip

* feature - fix syntax error in bash for if statement

* feature - skipping the installation of Adafruit_DHT - too much time

* feature - add avrdude & make temp file hidden

* feature - make sure the Dexter folder can be created & add REAMDE

* feature - make the same changes as on the grovepi

* feature - make the same changes to install.sh as on the GrovePi

* feature - fix lingering syntax issues

* feature - correct bad path to setup folder on gopigo

* feature - remove space echoes

* feature - yet again remove space echoes in install script

* feature - add source for 2 functions for avr installer

* feature - replacing "aborting" with "exiting"

* feature - check if script is run with pi user

* feature - correct name of installer

* feature - fix global variable issue

* feature - change how the installation script goes

* feature - slight grammatical change in readme

* feature - change sources

* rewrite Control Panel (#310)

* write Control Panel

* Fix GUI related to trimming

* feature - change script_tools reference (#311)

* write Control Panel

* Fix GUI related to trimming

* feature - change script_tools reference

* use direct links instead of redirects
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants