-
Notifications
You must be signed in to change notification settings - Fork 399
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
Physical Signal Range in XLSX Document #797
Comments
Hi @ganralf75, this should be possible with: |
thanks a lot @ebroecker! It works :-) |
Hi @ebroecker! I tried --additionalSignalAttributes min,max w/ arxml file and it seems that the min and max values are not plausible. Is it a bug or did I someting wrong? In this example signal.max should be 25.5barg kind regards, |
Hi @ganralf75 You could additionally add |
Hi @ebroecker, I added factor and offset. Both seem to be okay. I looks like the max is calculated from physical max (which is 25barg) and not from raw max (which is 250). |
Hi @ganralf75 Maybe this bug is fixed with https://github.com/ebroecker/canmatrix/pull/779/files - but is not fixed right? Maybe you could extract the contraints for this signal out of the ARXML, so we can isolate the problem ? |
Look like the physical min/max was calculated twice (initial value was calculated correctly), it would be better if we have a sample input to debug |
Hello,
I have a stupid question regarding generation of XLSX document out of a arxml / dbc.
I have considered that when a CAN signal contains a value table (e.g. w/ 2 values ERR & SNA) the physical range (e.g. 0...1000km) of the signal is not visible in XLSX document. Instead only the values of the value table will be documented.
Example w/ value table entries ERR & SNA but missing physical range in column T
Example w/ no signal value table defined but physical range is documented
Is there any parameter that I can use so that the physical signal range will also be documented in addition to the value table entries?
thanks in advance!
ganralf75
The text was updated successfully, but these errors were encountered: