Wrong data type in CostUpdatedPayload.total_cost in respect to json schema #557
Labels
bug
Something isn't working
Comments
|
Thanks for the report. You're right. The specification clearly states it's a decimal. And therefore, the |
Jared-Newell-Mobility
changed the title
|
PR created #596 |
Jared-Newell-Mobility
added a commit
that referenced
this issue
Feb 7, 2024
|
Merged so will now close |
Merged
Jared-Newell-Mobility
added a commit
that referenced
this issue
Feb 14, 2024
## 1.0.0-rc.1 (2024-02-14) - [#573](#573) Introduce Experimental Module For v2.1 - [#547](#547) Feat: Handle recursively serializing a dataclasses as a dictionary Thanks [@MacDue](https://github.com/MacDue) - [#601](#601) Fix case conversion for soc in non "State of Charge" context - [#523](#523) The serialisation of soc to SoC should not occur in camel case if it is existing at the beginning of a field - [#515](#515) Update Readthedocs configuration - [#602](#602) Correct v2g serialisation/deserialisation - [#557](#557) OCPP 2.0.1 Wrong data type in CostUpdated total_cost - [#564](#564) Add support For Python 3.11 and 3.12 - [#583](#583) OCPP v1.6/v2.0.1 deprecate dataclasses from calls and call results with the suffix 'Payload' - [#590](#336) snake_to_camel_case url to URL does not get converted correctly - [#591](#591) Camel_to_snake_case doesn't handle v2x correctly - [#593](#593) Update tests to use Call and CallResult without the suffix Payload - [#435](#435) Typo in CostUpdated Action - [#577](#577) v2.0.1 AttributeType Enum Corrections - [#340](#340) 2.0.1 dataclasses have a incorrect types that don't match carnality - [#519](#519) Typo in v201.enums.StatusInfoReasonType.invaild_schedule - [#510](#510) v2.0.1 UnitOfMeasureType - Enums missing and update docstring to allow use for variableCharacteristics - [#508](#508) Exception - OccurrenceConstraintViolationError doc string correction ## DEPRECATED ## - [#599](#599) v1.6 Action Enum members corrected IMPORTANT SEE UPGRADE PATH [#599](#599) - [#579](#579) v2.0.1 Action enums corrected - IMPORTANT SEE UPGRADE PATH [#579](#579) ## BREAKING ## - [#574](#574) Remove v1.6 deprecated enum members - IMPORTANT see upgrade path [#574](#574) - [#498](#498) Remove support for OCPP 2.0 - IMPORTANT SEE UPGRADE PATH [#498](#498)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello,
There is a mismatch between the data type for CostUpdatedPayload.total_cost in ocpp.ocpp.v201.datatypes and the json schema for CostUpdatedRequest. Specifically, in the first is defined as integer and the json schema a number, tha usually should be a float in python. Nevertheless, this difference yields the failure of the validation of the payload and hence the message is not sent. Though, just changing the type to float in CostUpdatedPayload.total_cost might not solve the problem, because afterwards the payload validation would fails when parsing the floats as metnioned in a previous issue of mine #555.
The text was updated successfully, but these errors were encountered: