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

Partner files for v1.0 usage service #299

Merged
merged 4 commits into from
Jun 11, 2023
Merged

Partner files for v1.0 usage service #299

merged 4 commits into from
Jun 11, 2023

Conversation

BuongiornoTexas
Copy link
Contributor

Scripts, configuration files and example dashboards to use the pwdusage micro-service in the Powerwall-Dashboard stack.

@jasonacox jasonacox merged commit 89db03f into jasonacox:main Jun 11, 2023
jasonacox added a commit that referenced this pull request Jun 11, 2023
@jasonacox
Copy link
Owner

Thanks @BuongiornoTexas !

I was not able to get this to build for armv7 (RPi 3) due to numpy build failure during pip install, but it seems to work fine for arm64 (Apple Silicon + RPi 4) and amd64.

I added my upload.sh script that I use to test and cross compile for architectures and upload to DockerHub.

@jasonacox
Copy link
Owner

Uploaded container jasonacox/pwdusage:1.0.0.

@BuongiornoTexas
Copy link
Contributor Author

I added my upload.sh script that I use to test and cross compile for architectures and upload to DockerHub.

Thanks for this - getting into multi-arch builds is definitely out of my comfort zone. I'll update the README to make the docker build an optional step.

I was not able to get this to build for armv7 (RPi 3) due to numpy build failure

I took a look around to see if I could find a solution to this and realised a) I don't think my docker install supports multiarch - at least not without additional work, so I can't test myself, and b) both numpy and pandas can have issues with installing into arm32/v7 docker images.

Unfortunately, I think sorting this one out is going to need someone who needs pwdusage for an RPi3 and is willing to spend the time resolving the problem. I'll raise it as an issue for pwdusage.

@jasonacox
Copy link
Owner

Sounds good! I agree on the armv7 approach.

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