Repository navigation
Sg90 servo - #165
Open
taartspi wants to merge 13 commits into
Open
Sg90 servo#165taartspi wants to merge 13 commits into
taartspi wants to merge 13 commits into
Conversation
…e device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
…e device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible. # Conflicts: # src/main/java/com/pi4j/devices/sg90/README.md # src/main/java/com/pi4j/devices/sg90/SG90_App.java
…e device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
…e device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible. # Conflicts: # src/main/java/com/pi4j/devices/sg90/SG90_App.java
… to resolve the errors loading the pi4j core 5.0.0-SNAPSHOT POM update. User text change remove pom raspberrypi new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
Collaborator
Author
|
ready for review. Will not build until the driver PR 183 is merged |
stefanhaustein
previously approved these changes
Oct 1, 2026
Duplicate what is in the merged code. This was out an modified trying to resolve the errors loading the pi4j core 5.0.0-SNAPSHOT POM update. User text change remove pom raspberrypi new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
…e and the Scanner Double methos also retuns a double. change float to double Duplicate what is in the merged code. This was out an modified trying to resolve the errors loading the pi4j core 5.0.0-SNAPSHOT POM update. User text change remove pom raspberrypi new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible. # Conflicts: # src/main/java/com/pi4j/devices/sg90/SG90_App.java
Change Double to double. The Double class parseDouble retuens a double and the Scanner Double methos also retuns a double. change float to double Duplicate what is in the merged code. This was out an modified trying to resolve the errors loading the pi4j core 5.0.0-SNAPSHOT POM update. User text change remove pom raspberrypi new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible.
stefanhaustein
previously approved these changes
Oct 8, 2026
update switch statement Change Double to double. The Double class parseDouble retuens a double and the Scanner Double methos also retuns a double. change float to double Duplicate what is in the merged code. This was out an modified trying to resolve the errors loading the pi4j core 5.0.0-SNAPSHOT POM update. User text change remove pom raspberrypi new example device SG90. Requires new driver build as I also added the device there. updated mcp472 5 and 8 to log level INFO. Needed so invalid parms details are visible. # Conflicts: # src/main/java/com/pi4j/devices/sg90/README.md
Collaborator
Author
|
my push stomped on your latest comment. my idea of a use case is you would start the app and be moving some arm back and forth and want to keep adjusting is. Something like that with the servo that rotates 360 continuously. |
stefanhaustein
approved these changes
Oct 9, 2026
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
New example device SG90 servo motor.
included several modifications to log level of existing files
The pom file is here as this was caught up in problems in the pi4j core snapshots not being available. It is now a copy of what pom is merged. Also, there are commented out lines that existed in the pom for a long time. I will to a separate PR to clean those out.