SwitchEV / RISE-V2G

The only fully-featured reference implementation of the Vehicle-2-Grid communication interface ISO 15118
MIT License
220 stars 92 forks source link

fix RISE ServiceDetail parameter according to standard #96

Open qiaozhou opened 1 year ago

qiaozhou commented 1 year ago

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.