Skip to content

narrow servo pulse range for safety; expose .fraction; improve doc #9

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

Merged
merged 1 commit into from
Jun 29, 2018

Conversation

dhalbert
Copy link
Contributor

Recent change of servo pulse limits to 550-2400 was too wide for some individual servos. Narrow the limits to 750-250, which provides about 135 degrees of rotation. Expound on this in additional documentation.

Expose the .fraction property for servos as another way the user can set the position, without worrying about possibly incorrect angles.

@dhalbert dhalbert requested a review from caternuson June 29, 2018 19:00
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Looks helpful! Thanks!

@tannewt tannewt merged commit c34abe2 into adafruit:master Jun 29, 2018
tannewt pushed a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jul 4, 2018
Updating https://github.com/adafruit/Adafruit_CircuitPython_AM2320 to 1.1.1 from 1.1.0:
  > add setup.py, add pypi to .travis.yml, misc. tweaks for pypi
  > add build artifacts to .gitignore

Updating https://github.com/adafruit/Adafruit_CircuitPython_APDS to 1.1.1 from 1.1.0:
  > Update apds9960_proximity_simpletest.py
  > Update apds9960_color_simpletest.py
  > setup.py: add adafruit-circuitpython-register to dependencies
  > add setup.py, add pypi to .travis.yml, misc. tweaks for pypi, .gitignore
  > Merge pull request adafruit/Adafruit_CircuitPython_APDS#8 from tannewt/fix_register_in_docs
  > Merge pull request adafruit/Adafruit_CircuitPython_APDS#7 from sommersoft/new_docs
  > Merge pull request adafruit/Adafruit_CircuitPython_APDS#6 from sommersoft/new_docs

Updating https://github.com/adafruit/Adafruit_CircuitPython_AS726x to 1.0.2 from 1.0.1:
  > .travis.yml: add missing `pip` invocation
  > add setup.py, add pypi to .travis.yml, misc. tweaks for pypi, .gitignore

Updating https://github.com/adafruit/Adafruit_CircuitPython_Crickit to 2.0.1 from 1.0.2:
  > change pypi to know it's a single-file library
  > Merge pull request adafruit/Adafruit_CircuitPython_Crickit#7 from dhalbert/rework_api
  > Merge pull request adafruit/Adafruit_CircuitPython_Crickit#6 from dhalbert/rework_api
  > Merge pull request adafruit/Adafruit_CircuitPython_Crickit#5 from dhalbert/rework_api

Updating https://github.com/adafruit/Adafruit_CircuitPython_Se to 1.2.2 from 1.2.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_Se#17 from dhalbert/fix-pin-mapping-AGAIN

Updating https://github.com/adafruit/Adafruit_CircuitPython_Motor to 1.2.0 from 1.1.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_Motor#10 from dhalbert/check_fraction_range
  > Merge pull request adafruit/Adafruit_CircuitPython_Motor#9 from dhalbert/better_servo_limits
@dhalbert dhalbert deleted the better_servo_limits branch April 7, 2022 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants