Skip to content
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

Create pulse funcition #78

Merged
merged 10 commits into from
Jul 3, 2015
Merged

Create pulse funcition #78

merged 10 commits into from
Jul 3, 2015

Conversation

gfth4x07
Copy link
Contributor

@gfth4x07 gfth4x07 commented Jun 3, 2015

Just a function to pulse a pin. This function will take easy the serial communication and write in a Shift-Register.
I tested in a Raspberry Pi 2 B and it's works.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.27%) to 59.11% when pulling e705f0b on gfth4x07:master into 628d39f on pingo-io:master.

@gfth4x07
Copy link
Contributor Author

Now Pingo supports GrovePI Board.
I have tested!

@Vido
Copy link
Contributor

Vido commented Jul 1, 2015

@gfth4x07, thanks for your time.

Please, fix the flake8 warnings.

0.45s$ flake8 --ignore=E501 pingo
pingo/rpi/__init__.py:4:34: W292 no newline at end of file
pingo/rpi/grove.py:11:13: F401 'grovepi' imported but unused
pingo/rpi/grove.py:54:67: E226 missing whitespace around arithmetic operator
pingo/rpi/grove.py:66:52: E226 missing whitespace around arithmetic operator

I will point them in the comments...

@gfth4x07
Copy link
Contributor Author

gfth4x07 commented Jul 1, 2015

I think it's better import grovepi as grovepi. The libraries name is grovepi why not use this name?

@gfth4x07 gfth4x07 closed this Jul 1, 2015
@gfth4x07 gfth4x07 reopened this Jul 1, 2015
@Vido
Copy link
Contributor

Vido commented Jul 1, 2015

@gfth4x07 ok...
galileo.py does it the same way...

Just add a dirty fix:

import grovepi as grovepi  # noqa

Vido added a commit that referenced this pull request Jul 3, 2015
Thanks to @gfth4x07, now Pingo has GrovePi.
@Vido Vido merged commit 265a6d6 into pingo-io:master Jul 3, 2015
@Vido
Copy link
Contributor

Vido commented Jul 3, 2015

Thanks to @gfth4x07, now Pingo has GrovePi.

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.

None yet

3 participants