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.

Sign up for GitHub

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

Jump to bottom

Ranges for Float and Double #705

Closed
erikbosch opened this issue Jan 25, 2024 · 1 comment · Fixed by #716
Closed

Ranges for Float and Double #705

erikbosch opened this issue Jan 25, 2024 · 1 comment · Fixed by #716

Comments

@erikbosch
Copy link
Collaborator

erikbosch commented Jan 25, 2024 •
edited
Loading

What is the source for float/double range at https://covesa.github.io/vehicle_signal_specification/rule_set/data_entry/data_types/

Ranges are possible different compared to AUTOSAR


float | floating point number | -3.4e -38 | 3.4e 38
double | double precision floating point number | -1.7e -300 | 1.7e 300

Not fully compatible with for example:

IEEE 754 32-bit base-2 floating-point
https://en.wikipedia.org/wiki/Double-precision_floating-point_format

Shall we possibly base limits on IEEE754 binary 32 and binary64
@erikbosch
Copy link
Collaborator Author

MoM:

  • Nick: What be good to align with standards
  • Sebastian: Reference to IEEE 754 would not be bad. maybe also endian issues. maybe refer to IEEE 754 limits

erikbosch added a commit to boschglobal/vehicle_signal_specification that referenced this issue Feb 8, 2024
@erikbosch
Update documentation
d6a7cd1 
Clarifies string/float/double.
Reformat recommendatations to state that they also are rules for VSS catalog
Remove enum recommendation

Fixes COVESA#705
Fixes COVESA#706
Fixes COVESA#700

Signed-off-by: Erik Jaegervall <erik.jaegervall@se.bosch.com>
@erikbosch erikbosch mentioned this issue Feb 8, 2024
Merged
erikbosch added a commit to boschglobal/vehicle_signal_specification that referenced this issue Feb 9, 2024
@erikbosch
Update documentation
ca75887 
Clarifies string/float/double.
Reformat recommendatations to state that they also are rules for VSS catalog
Remove enum recommendation

Fixes COVESA#705
Fixes COVESA#706
Fixes COVESA#700

Signed-off-by: Erik Jaegervall <erik.jaegervall@se.bosch.com>
erikbosch added a commit to boschglobal/vehicle_signal_specification that referenced this issue Feb 9, 2024
@erikbosch
Update documentation
ef04ed7 
Clarifies string/float/double.
Reformat recommendatations to state that they also are rules for VSS catalog
Remove enum recommendation

Fixes COVESA#705
Fixes COVESA#706
Partially solves COVESA#700
Fixes COVESA#662

Signed-off-by: Erik Jaegervall <erik.jaegervall@se.bosch.com>
erikbosch added a commit to boschglobal/vehicle_signal_specification that referenced this issue Feb 14, 2024
@erikbosch
Update documentation
fdd5b6b 
Clarifies string/float/double.
Reformat recommendatations to state that they also are rules for VSS catalog
Remove enum recommendation

Fixes COVESA#705
Fixes COVESA#706
Partially solves COVESA#700
Fixes COVESA#662

Signed-off-by: Erik Jaegervall <erik.jaegervall@se.bosch.com>
erikbosch added a commit that referenced this issue Feb 20, 2024
@erikbosch
Update documentation
93550c3 
Clarifies string/float/double.
Reformat recommendatations to state that they also are rules for VSS catalog
Remove enum recommendation

Fixes #705
Fixes #706
Partially solves #700
Fixes #662

Signed-off-by: Erik Jaegervall <erik.jaegervall@se.bosch.com>
erikbosch added a commit to boschglobal/vehicle_signal_specification that referenced this issue May 6, 2024
@erikbosch
Update documentation
db3b08b 
Clarifies string/float/double.
Reformat recommendatations to state that they also are rules for VSS catalog
Remove enum recommendation

Fixes COVESA#705
Fixes COVESA#706
Partially solves COVESA#700
Fixes COVESA#662

Signed-off-by: Erik Jaegervall <erik.jaegervall@se.bosch.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update documentation boschglobal/vehicle_signal_specification
1 participant
@erikbosch