-
Notifications
You must be signed in to change notification settings - Fork 23
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
Hog upload fix #897
base: main
Are you sure you want to change the base?
Hog upload fix #897
Conversation
Eco-CI Output:
🌳 CO2 Data: |
[0] https://github.com/green-coding-solutions/hog/blob/ea2f5970b8281650ac21687613b5ce098ddf8de4/power_logger.py#L180 |
@ribalba Can you please close this and move code to the EE repository |
Also please include any lingering HOG fixes we have around. I remember you posted somewhere refactorings to the object_specification.py by moving all checks for HOG data to pydantic |
This is currently a draft PR to make the PowerHog data upload usable again.
It was deactivated as we were seeing regular errors.
It has now been deactivated with some learnings. More are to come in this PR.
Learnings
GET /v1/hog/add
. Since we monitor 404s this was clogging up our logs. I have deactivated the route for now ... but why is this happening?