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

fix RISE ServiceDetail parameter according to standard #96

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qiaozhou
Copy link

@qiaozhou qiaozhou commented Aug 11, 2023

DESCRIPTION

During test of communication with vehicule that require VAS (value added services) it needs Internet connection to access remote services.

We noticed the ServiceDetailsRes EXI packet is not correct, and do not match the standard ISO15118 description.

V2G2-348 The massage element shall be used as defined in table 85

Especialy the sentence in table 85: Only one for each parameter can be selected,

Moreover the V2G message exemple 8 ServiceDetailsRes message (page 279) in standard 15118-2:2014(E)
shows clearly the multiple occurence of parameter fields for each protocol and port which justify the current fix.

<v2gci_t:ParameterSet>
    <v2gci_t:ParameterSetID>3</v2gci_t:ParameterSetID>
    <v2gci_t:Parameter v2gci_t:Name="Protocol">
        <v2gci_t:stringValue>HTTP</v2gci_t:stringValue>
    </v2gci_t:Parameter>
    <v2gci_t:Parameter v2gci_t:Name="Port">
        <v2gci_t:intValue>80</v2gci_t:intValue>
    </v2gci_t:Parameter>
</v2gci_t:ParameterSet>
<v2gci_t:ParameterSet>
    <v2gci_t:ParameterSetID>4</v2gci_t:ParameterSetID>
    <v2gci_t:Parameter v2gci_t:Name="Protocol">
        <v2gci_t:stringValue>HTTPS</v2gci_t:stringValue>
    </v2gci_t:Parameter>
    <v2gci_t:Parameter v2gci_t:Name="Port">
        <v2gci_t:intValue>443</v2gci_t:intValue>
    </v2gci_t:Parameter>
</v2gci_t:ParameterSet>

Although RISE is no more supported, please find this pull request as participation to the community and help to make EV charger more compatible with standard.

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.

2 participants