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

simplify / restructure code #15

Open
jinjirosan opened this issue Sep 5, 2017 · 3 comments
Open

simplify / restructure code #15

jinjirosan opened this issue Sep 5, 2017 · 3 comments

Comments

@jinjirosan
Copy link
Owner

def sensors

@jinjirosan jinjirosan created this issue from a note in Baby Carriage Monitor - the 2.0 (Backlog) Sep 5, 2017
@jinjirosan jinjirosan self-assigned this Sep 5, 2017
@jinjirosan jinjirosan added this to the First base milestone Sep 5, 2017
@jinjirosan
Copy link
Owner Author

Refactor everything after mvdwrd session :)

  • for each sensor separate function including exception handling. Return values should be clean.
  • one function to gather all sensor data into dictionary. Cleanup values correctly.
  • define comm function for output to dweet.io and initialstate.com
  • define simple main function to call the dictionary and comm function
  • end with if name == "main":

@jinjirosan jinjirosan moved this from Backlog to Selected for development in Baby Carriage Monitor - the 2.0 Sep 5, 2017
@jinjirosan jinjirosan moved this from Selected for development to In Progress in Baby Carriage Monitor - the 2.0 Sep 15, 2017
@jinjirosan
Copy link
Owner Author

REFACTOR branch started

@jinjirosan
Copy link
Owner Author

Moved all sensor data related to weather conditions into one class. Moved all transmit code into one class, still sending to dweet.io, intitialstate.com but also need to send to centreon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

1 participant