8000 Update sax.yaml by oekinger · Pull Request #21277 · evcc-io/evcc · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update sax.yaml #21277

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 18, 2025
Merged

Update sax.yaml #21277

merged 1 commit into from
May 18, 2025

Conversation

oekinger
Copy link
Contributor

Please change the port used in the template becausre it is wrong: must be 502 instead of 3600.
Thanks,
Marko

Update sax.yaml
ecc92d3 
The port used in the template is wrong: must be 502 instead of 3600
sourcery-ai[bot]
sourcery-ai bot reviewed May 17, 2025
View reviewed changes
Copy link
Contributor
@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @oekinger - I've reviewed your changes and they look great!

Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good
Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@oekinger
Copy link
Contributor Author

Perhaps this change is depending on the Revision of the sax battery: I have a 2024 produced 7,7 kWh battery. Perhaps someone can verify if his/her sax works with port 3600

@andig andig added the devices Specific device support label May 17, 2025
@andig
Copy link
Member
andig commented May 17, 2025

/cc @Sebreiter

@Sebreiter
Copy link

I have a 2023 produced5.2 kWh battery. My sax works with port 3600.

  • name: grid
    type: template
    template: sax
    usage: grid

Modbus TCP

modbus: tcpip
id: 64
host: xxx.xxx.xxx.xx
port: 3600

@oekinger
Copy link
Contributor Author

Ok, then we have a firmware dependency for the modbus port. Perhaps it’s enough to mention this in the description of the template?

@andig
Copy link
Member
andig commented May 18, 2025

I guess the newer products might use the default port, so let's change it.

@andig andig merged commit ffb53c0 into evcc-io:master May 18, 2025
6 checks passed
guido4096 pushed a commit to guido4096/evcc that referenced this pull request May 25, 2025
@oekinger @guido4096
Sax: update port (evcc-io#21277)
19666e8
@juergen-weber
Copy link
Contributor
juergen-weber commented Jun 8, 2025 •
edited
Loading

To all you guys using a SAX Home with evcc:
The sax.yaml template won't work correctly due to 2 reasons:

  • the SAX modbus implementation resets all settings at on unpredictable interval to default values. So to make settings work for a longer time (EV charging without discharging the home battery) the command must be repeated every minute or so.
  • writing to the SAX battery requires the use of "writemultiple" instead of "writesingle". Unfortunately, I have currently no EV at home - so I can't test it. Executing function code 06 (writesingle) on register 43 gets me an error, while function code 10h works.

I'm not familiar with github, so I'm not able to push the following snippet. Maybe someone of you guys can do this:

  batterymode:
    source: watchdog
    timeout: 60s
    reset: 1 # reset watchdog on normal
    set:
      source: switch
      switch:
      - case: 1 # normal
        set:
          source: const
          value: 4600 # stop
          set:
            source: modbus
            {{- include "modbus" . | indent 10 }}
            register:
              address: 43 # 2Bh battery discharging power [W]
              type: writemultiple
              encoding: uint16
      - case: 2 # hold
        set:
          source: sequence
          set:
          - source: const
            value: 0 # discarging power [W]
            set:
              source: modbus
              {{- include "modbus" . | indent 12 }}
              register:
                address: 43 # 2Bh battery discharging power [W]
                type: writemultiple
                encoding: uint16
          - source: const
            value: 0 # W
            set:
              source: modbus
              {{- include "modbus" . | indent 12 }}
              register:
                address: 43 # 2Bh battery discharging power [W]
                type: writemultiple
                encoding: uint16

Regards Jürgen

@andig
Copy link
Member
andig commented Jun 8, 2025 •
edited
Loading

so I'm not able to push the following snippet

Can you please apply formatting? To edit, just open the file here in github and add your changes. Of course, these need be tested.

@juergen-weber
Copy link
Contributor

Code snippet is now formatted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sourcery-ai sourcery-ai[bot] sourcery-ai[bot] left review comments

Assignees

@premultiply premultiply

Labels
devices Specific device support
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@oekinger @andig @Sebreiter @juergen-weber @premultiply
0