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

Feature/evse #6

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

mmb-davidsmith
Copy link
Contributor

EVSE work

EVSEManagement.xml Outdated Show resolved Hide resolved
<field name="EvseState" type="EvseState" />
<!-- Event Data Should Really Try To Follow The Pattern In OTA unless there's something wrong with that pattern. -->
<!-- That would allow use to better capture the field formats for the content as separate fields -->
<field name="EventData" type="octstr" />
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would highly suggest taking a look at how this is handled in the Image Block Response in OTA. The XML schema allows us to specify a presentIf value for fields which would allow us to make certain fields only present based on EvseState.

Choose a reason for hiding this comment

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

Question/proposal made separately

Copy link
Contributor

Choose a reason for hiding this comment

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

is it solved ?

EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Show resolved Hide resolved
EVSEManagement.xml Show resolved Hide resolved
<field name="EvseState" type="EvseState" />
<!-- Event Data Should Really Try To Follow The Pattern In OTA unless there's something wrong with that pattern. -->
<!-- That would allow use to better capture the field formats for the content as separate fields -->
<field name="EventData" type="octstr" />

Choose a reason for hiding this comment

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

Question/proposal made separately

EVSEManagement.xml Outdated Show resolved Hide resolved
@iwinterburn iwinterburn marked this pull request as ready for review September 24, 2022 08:48
EVSEManagement.xml Outdated Show resolved Hide resolved
@iwinterburn
Copy link

Client command IDs updated as requested

<attribute id="000D" name="CircuitCapacity" type="uint16" />

<!-- Attribute Set 0x01: Reading Information -->
<attribute id="0100" name="ChargingVoltage" type="uint16" required="true" max="65535" />
Copy link
Contributor

Choose a reason for hiding this comment

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

So it need to be added here ? on this line ?
What is missing to get this PR merged @iwinterburn ?

Copy link

@iwinterburn iwinterburn left a comment

Choose a reason for hiding this comment

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

Some attribute default values agreed on Sept 27 BG call.
Clinet Command IDs renumbered to ovoid overlap with Server command (0x01)

EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
EVSEManagement.xml Outdated Show resolved Hide resolved
mmb-davidsmith and others added 4 commits December 22, 2022 13:32
… There are a number of outstanding questions which will be submitted for the spec authors to review. Further adjustments may be needed before this is ready to be merged into the primary work stream.
Updates the default value for all Multipliers & Divisors to 1
Renumbers the commands received by the server.
@jmeg-sfy
Copy link
Contributor

Hello i did an update to rebase + fixe Ian comments

@jmeg-sfy
Copy link
Contributor

@mmb-davidsmith I took the liberty to rebase force push into your branch

@michaelleecowan
Copy link

michaelleecowan commented Dec 22, 2022 via email

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

Successfully merging this pull request may close these issues.

4 participants