ocpp icon indicating copy to clipboard operation
ocpp copied to clipboard

Feature: Support of multiple connectors

Open ernst77 opened this issue 3 years ago • 4 comments

This will provide support for chargers with multiple connectors. As mentioned in #796 #801 #594 #792 #798

Thanks for the code https://github.com/jthun, I've made some modifications and put it up to date with the current code.

If the separation of connectors is not needed when connector count < 2. Let me know We could add if and merge all properties to charger how it used to be done.

ernst77 avatar Mar 22 '23 09:03 ernst77

(This is my first proper interaction in this repo, so please feel free to ignore.)

It looks like you're making good progress in multiple connector support. My main question is can you get the NumberOfConnectors from the charge point (getConfiguration), so you don't have to set it manually? I guess this would probably be a total pain, so it might be worth leaving out?

Jallerton00 avatar Apr 23 '23 18:04 Jallerton00

@ernst77 There are still some linting issues, and a test failure. These need to be fixed before I can merge the changes.

lbbrhzn avatar May 13 '23 20:05 lbbrhzn

Stale pull request message

github-actions[bot] avatar Sep 19 '23 06:09 github-actions[bot]

Hey! Are linting errors and tests (and now merge conflicts) the only thing stopping this from getting merged? Does the feature itself work? Asking to set expectations and see if I can help as well.

lightheaded avatar Oct 24 '24 15:10 lightheaded

We have a rather large update pending (support for ocpp v2), better wait until that has been merged to main.

lbbrhzn avatar Nov 07 '24 08:11 lbbrhzn

I'll hold off, thanks for sharing the message here as well. I'll meanwhile get more familiar with the protocol and try and see if I can look at some traffic that is happening currently between the server and the charger.

The main question still stands, though - besides the hygiene (tests, linting etc.), did the PR actually manage to get multiple chargers to work properly?

lightheaded avatar Nov 07 '24 09:11 lightheaded

@ernst77, lot's of things changed in order to support PCPP 2.0. Can you rebase your changes / resolve conflicts?

lbbrhzn avatar Nov 25 '24 14:11 lbbrhzn

I'll hold off, thanks for sharing the message here as well. I'll meanwhile get more familiar with the protocol and try and see if I can look at some traffic that is happening currently between the server and the charger.

The main question still stands, though - besides the hygiene (tests, linting etc.), did the PR actually manage to get multiple chargers to work properly?

I have multiple connector support running, in the code that @ernst77 started his work from, so his code should work. There are a few glitches in my code but it works acceptably well for my daily use. I think this is where @ernst77 started: https://github.com/jthun/ocpp/tree/feature/multiple_connectors, but he did a lot of good things, so better start there.

jthun avatar Dec 11 '24 12:12 jthun

Hi all, apologies for necro-ing this thread, slowly working my way through the different issues/branches/PRs/forks associated with this topic. To be clear, the current state (as of 0.9.0) is that dual connector chargers (such as the CTEK Chargestorm 2 that I also have) are not supported via the "main" OCPP HACS integration, but was working (albeit with a few bugs as reported by @jthun) in an older fork by @ernst77 that ended up in this abandoned PR? Just wondering if I can grab a copy of this older fork into HA for the meantime to have something working full stop... Obviously the ideal would be to refactor the older changes on top of 0.9.0 and re-attempt the merge, but I'm not sure I have the time or the expertise to tackle this at the moment, so just trying to get a minimum viable product working instead!

sftgunner avatar Aug 12 '25 22:08 sftgunner