[LKuemmel]

No description provided.

[Copilot]

Pull request overview

Adds a new Solarfocus “Vampair” consumer integration (Modbus TCP-based) and aligns a few existing consumers/sample code with recent API/config changes.

Changes:

• Introduce packages/modules/consumers/solarfocus/ vendor descriptor and a new vampair consumer + configuration.
• Fix my-PV Elwa-E limiting logic (clamp to >= 0) and correct nonlocal usage.
• Update generic JSON consumer config to use jq_currents tuple and adjust JSON consumer action invocations; update sample consumer import.

Reviewed changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/modules/consumers/solarfocus/vendor.py	Adds vendor descriptor for the new Solarfocus consumer family.
packages/modules/consumers/solarfocus/vampair/consumer.py	Implements the Vampair consumer using Modbus TCP and sim counting.
packages/modules/consumers/solarfocus/vampair/config.py	Adds configuration/setup model for Vampair consumer.
packages/modules/consumers/solarfocus/vampair/init.py	Package marker for the new consumer module.
packages/modules/consumers/solarfocus/init.py	Package marker for the new vendor namespace.
packages/modules/consumers/my_pv/elwa_e/consumer.py	Fixes power-limit clamping and nonlocal bindings.
packages/modules/consumers/generic/json/consumer.py	Adjusts JSON consumer action calls to match current helper function patterns.
packages/modules/consumers/generic/json/config.py	Consolidates current filters into jq_currents tuple.
docs/samples/sample_consumer/sample_consumer/consumer.py	Updates sample consumer import (currently to an invalid module path).

Suppressed comments (2)

packages/modules/consumers/generic/json/consumer.py:85

• Same issue as switch_on(): the no-op function returned when jq_switch_off is unset won’t accept params=.... Pass the payload positionally to avoid TypeError when this feature isn’t configured.

        jq_switch_off(params={"state": False})

packages/modules/consumers/generic/json/consumer.py:89

• When jq_set_power_limit is not configured, create_post_function() returns a no-op lambda that won’t accept the keyword argument params=.... Pass the payload dict positionally so the call remains a no-op instead of raising TypeError.

        jq_set_power_limit(params={"power_limit": power_limit})

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.