-
Notifications
You must be signed in to change notification settings - Fork 17
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
add json outpuy #24
add json outpuy #24
Conversation
@rosswarren can evaluate? |
epevermodbus/command_line.py
Outdated
output["battery_discharge"] = controller.get_battery_discharge() | ||
output["battery_charge"] = controller.get_battery_charge() | ||
output["charging_mode"] = controller.get_charging_mode() | ||
print(output) |
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.
Please use the builtin json.dumps(output)
here.
Using a library we make sure that the json will always be well-formatted and can be read across platforms.
The Python str()
output of a dict should not be considered stable.
pr updated |
@sante85 Thx! LGTM |
when you publish on pypi.org?
thanks
Il giorno dom 24 dic 2023 alle ore 13:42 SmithChart <
***@***.***> ha scritto:
… @sante85 <https://github.com/sante85> Thx! LGTM
—
Reply to this email directly, view it on GitHub
<#24 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADLHTN6TIKIYB5T4Z2KMHZTYLAPJRAVCNFSM6AAAAABA6VHHNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYGUYDQMJSGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
hi Rosswarren/Epevermodbus,
when you think to publish?
thanks
Il giorno dom 24 dic 2023 alle ore 17:03 Sante Stanisci ***@***.***>
ha scritto:
… when you publish on pypi.org?
thanks
Il giorno dom 24 dic 2023 alle ore 13:42 SmithChart <
***@***.***> ha scritto:
> @sante85 <https://github.com/sante85> Thx! LGTM
>
> —
> Reply to this email directly, view it on GitHub
> <#24 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ADLHTN6TIKIYB5T4Z2KMHZTYLAPJRAVCNFSM6AAAAABA6VHHNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYGUYDQMJSGQ>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
|
@SmithChart @rosswarren @hrford you are here to merge PR? thanks |
@sante85 Sorry, I am just a contributor and can not merge. |
Why the owner not merge?
Il ven 5 gen 2024, 19:20 SmithChart ***@***.***> ha scritto:
… @sante85 <https://github.com/sante85> Sorry, I am just a contributor and
can not merge.
—
Reply to this email directly, view it on GitHub
<#24 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADLHTN6MVH3LF2OCFR3WWWTYNA75TAVCNFSM6AAAAABA6VHHNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZGA4DKMJTGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
i have update, @rosswarren @rafael2k @hrford please merge |
thank you, these changes have been incorporated into #26 so closing this PR |
No description provided.