-
Notifications
You must be signed in to change notification settings - Fork 69
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
v4.0.0 - Add Cloud Mode support via pyPowerwall v0.7.0 #414
Merged
Changes from 2 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
a0ed67c
Stage updates for pypowerwall v0.7.0
jasonacox 1ab975e
v4.0.0
jasonacox 39462a1
Add auth and site file creation
jasonacox ad1958c
Add .auth folder
jasonacox 252f335
Update powerwall.yml for .auth
jasonacox d5bc73e
Add PW_AUTH_PATH to Docker Compose config
mcbirse 7c54343
Add CQ for nominal_full_pack_energy
jasonacox a2fff22
Merge branch 'v4.0.0' of https://github.com/jasonacox/Powerwall-Dashb…
jasonacox 5aa5fcd
Capture SIGTERM when weather411 halts due to missing config
mcbirse e8817c5
Add auth files
mcbirse 32b946f
Remove profiles
mcbirse 666947a
Update setup scripts and remove profiles
mcbirse b9bd555
Update container versions and remove profiles
mcbirse 7e1bd00
Update pypowerwall
jasonacox 90c2d7e
Add v3.0.8
jasonacox f41de0d
v4.0.0 upgrade script
mcbirse fde66b2
Fix network scan ip addr in Windows Git Bash
mcbirse 7e7d692
Merge branch 'main' into v4.0.0
jasonacox 1d96c9a
Merge branch 'v4.0.0' of https://github.com/jasonacox/Powerwall-Dashb…
jasonacox fa1bd89
Get ip addr based on OS
mcbirse 1b2aae4
Fix for missing cmd
mcbirse 94c75f7
Update verify script
mcbirse 253dffe
Add sample docker compose for tesla-history
mcbirse 2e1c504
Update setup options, README and RELEASE notes
mcbirse 2f58551
Update to latest pypowerwall and proxy
mcbirse a493e91
Update README
jasonacox 910e627
Add animation to solar-only dashboard
jasonacox 7807a8c
Add script descriptions
jasonacox File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't work on my Mac. I can use
ipconfig getifaddr en0
orifconfig | grep 'inet ' | grep -v '127.0.0.1' | awk '{print $2}'
.maybe something like this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks better than what I did. I will incorporate this or similar into the script - feel free to change it though.
I was trying to find a consistent OS independent way of discovering the local IP address in bash.... it seems there isn't one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good! Yes, search for one and even asked ChatGPT! 😂 No luck. Worse case, if it gets an empty or wrong, response, the scan won't rung and the user can manually enter the IP during setup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep true. When you think something should be simple... lol
To complicate things, small change for this I'll remove the "32" in
CYGWIN*|MINGW32*|MSYS*)
as my Git Bash returns MINGW64 😆There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good and we are getting close @mcbirse. What left on your list? I'll start testing.