-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[modbus][sunspec] Fixed type and scale errors on acc32 fields #7826
Conversation
ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
Travis tests were successfulHey @mrbig, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I wonder why it doesn't list you as codeowner of the files in this PR... |
Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
Fixed the alphabetic order. Seems like to me that this fixed it. |
Travis tests were successfulHey @mrbig, |
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu> Signed-off-by: CSchlipp <christian@schlipp.de>
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu> Signed-off-by: Daan Meijer <daan@studioseptember.nl>
…b#7826) * [modbus][sunspec] Fixed type and scale errors on acc32 fields ACC32 fields were mistakenly parsed as unsigned int instead of signed default scale value is 10^0=1 now scaling was missing for 10^1 fields * [modbus][sunspec] Changed CODEOWNERS order Seems like mrbig is not recognized as codeowner by github. Trying to fix the alphabetic order to see if this helps Signed-off-by: Nagy Attila Gabor <mrbig@sneaker.hu>
This PR fixes some scaling errors in the modbus.sunspec bundle: