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

Incorrect boost times shown with zappi #19

Open
plord12 opened this issue Oct 27, 2023 · 3 comments
Open

Incorrect boost times shown with zappi #19

plord12 opened this issue Oct 27, 2023 · 3 comments

Comments

@plord12
Copy link

plord12 commented Oct 27, 2023

With my zappi and no scheduled boosts set, the script returns invalid data -

myenergi zappi show
...
Boost with -1kWh
Smart Boost start at 17:-1 add 5kWh
Minimum green level: 100%

In this case API response from cgi-boost-time was -

{
    "boost_times": [
        {
            "bdd": "00000000",
            "bdh": 0,
            "bdm": 0,
            "bsh": 0,
            "bsm": 0,
            "slt": 11
        },
        {
            "bdd": "00000000",
            "bdh": 0,
            "bdm": 0,
            "bsh": 0,
            "bsm": 0,
            "slt": 12
        },
        {
            "bdd": "00000000",
            "bdh": 0,
            "bdm": 0,
            "bsh": 0,
            "bsm": 0,
            "slt": 13
        },
        {
            "bdd": "00000000",
            "bdh": 0,
            "bdm": 0,
            "bsh": 0,
            "bsm": 0,
            "slt": 14
        }
    ]
}
@plord12
Copy link
Author

plord12 commented Oct 27, 2023

even with a boost time set, I see the same -

myenergi zappi show
...
Boost with -1kWh
Smart Boost start at 17:-1 add 5kWh
Minimum green level: 100%

cgi-boost-time returned -

{
    "boost_times": [
        {
            "bdd": "00000100",
            "bdh": 1,
            "bdm": 0,
            "bsh": 23,
            "bsm": 0,
            "slt": 11
        },
        {
            "bdd": "00000000",
            "bdh": 0,
            "bdm": 0,
            "bsh": 0,
            "bsm": 0,
            "slt": 12
        },
        {
            "bdd": "00000000",
            "bdh": 0,
            "bdm": 0,
            "bsh": 0,
            "bsm": 0,
            "slt": 13
        },
        {
            "bdd": "00000000",
            "bdh": 0,
            "bdm": 0,
            "bsh": 0,
            "bsm": 0,
            "slt": 14
        }
    ]
}

@jwillemsen
Copy link
Contributor

Looks the cli.py doesn't use the fetch_boost_data as part of zappi.py, it uses some other fields from the returned data

@jwillemsen
Copy link
Contributor

#41 has made a first step, now the data must be used

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

No branches or pull requests

2 participants