-
Notifications
You must be signed in to change notification settings - Fork 192
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
Bug/line follower + overall mutex #291
Merged
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
RobertLucian
approved these changes
Mar 2, 2018
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
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.
two fixes for the price of one.
The import doesn't work the way it used to within conda. This approach should be more universal.