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

feat: add debug sensor for development workflow on hardware that does… #176

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

Conversation

sfishel18
Copy link

…n't support any existing sensors

@sfishel18
Copy link
Author

👋 hello! i came across your product searching for open source projects working to fight climate change. i'd love to contribute if i can.

the first challenge i had trying out the code base was that i don't have easy access to bare-metal Linux. i do development on a Mac and generally deploy things on GCP. so i had the thought to get some experience with the code by introducing a "debug" sensor that would just output synthetic power consumption information without requiring the underlying hardware to be compatible with the powercap/rapl sensor.

it's turning into a larger refactor than i intended, so i thought it would be best to pause where it is and put it up for discussion:

  • is this functionality generally useful to more than just me?
  • is this the right approach to adding it?

also, i'm new to Rust, so any feedback on code style and patterns is very welcome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

2 participants