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

Improved esp8266 clients #29

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Improved esp8266 clients #29

wants to merge 4 commits into from

Conversation

XayOn
Copy link

@XayOn XayOn commented Feb 20, 2017

Still a WIP, but it's growing quickly and seems more stable than the current clients:

  • Deleted firmware.
  • Added references to actual documentations on
    flashing and file uploading
  • Added a makefile
  • Made clients configurable and modular, with
    a robust init system.
  • Deleted virtualenv (should not be distributed this way)

It still lacks:

  • More card types (rgb sliders is missing from the original lua examples)
  • Better documentation on how to configure them (it's intuitive once you open config/ directory, but still...)

- Deleted firmware.
- Added references to actual documentations on
flashing and file uploading
- Added a makefile
- Made clients configurable and modular, with
a robust init system.
- Deleted virtualenv (should not be distributed this way)
Still not working correctly...
@edfungus edfungus self-assigned this Feb 26, 2017
@edfungus
Copy link
Owner

edfungus commented Mar 2, 2017

@XayOn Is there a reason for the switch to a rst format readme?

@XayOn
Copy link
Author

XayOn commented Mar 2, 2017

Not a reasonable one apart that I dont really know markdown, sorry.

You can change it backup but Igual tough important yo update the Docs.

Anyway, i've recently implemented pasive sensors and im unable to make them work so this implementation IS probably not ok.

Could you please check It ir document the inbox/outbox/deviceinfo part?
It seems like when i add a loop to update sensor data each X secs i loose the hability to send deviceinfo and therefore the devolver fails yo Connect...

@edfungus
Copy link
Owner

Considering

Anyway, i've recently implemented pasive sensors and im unable to make them work so this implementation IS probably not ok.

Are there going to be improvements to this pull request? I am uncertain about the state of this pull request.

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