Use more concise data types for deserialization

[mburtscher]

Currently most of the data is deserialized with either string, int and float. However there are several data points which could use much more concise data types such as TimeSpan, bool or custom implementations for Speed, Distance etc.

I have created some custom type converters for YamlDotNet in my SDK project:
https://github.com/OpenIRacingTools/Sdk/tree/master/OpenIRacingTools.Sdk/Yaml

They can be registered in YamlDotNet using WithTypeConverter() method:
https://github.com/OpenIRacingTools/Sdk/blob/master/OpenIRacingTools.Sdk/Sdk.cs#L55

[mburtscher]

There is also a lot of information which can be expressed using proper enum types:

https://github.com/OpenIRacingTools/Sdk/tree/master/OpenIRacingTools.Sdk/Model/Enums

The code mentioned above contains an EnumTypeConverter which will read string representation for serialization of enum values from the Display attribute like in WeatherType.

[LuckyNoS7evin]

Yes, this was/is on the roadmap, something "to be done"

[LuckyNoS7evin]

@mburtscher if you want to do this could you do some testing on serialization times?

I'm of the opinion that we should be leaving it down to the developer to do what they want with the data. Now the reason for the serialization module was to just give the "basics" and therefore the basic types that iRacing gives out.

So if they give you a float it should be serialized as a float.

There is a Client separate from this SDK which may be the place to do this https://github.com/SlevinthHeaven/s7evemetry

[mburtscher]

@LuckyNoS7evin I'm sorry for not responding. I don't think that I will have the time to work on any of this the coming weeks/months. So feel free to close if you don't feel like it was a significant improvement.