{"action":"created","issue":{"url":"https://api.github.com/repos/openETCS/modeling/issues/629","labels_url":"https://api.github.com/repos/openETCS/modeling/issues/629/labels{/name}","comments_url":"https://api.github.com/repos/openETCS/modeling/issues/629/comments","events_url":"https://api.github.com/repos/openETCS/modeling/issues/629/events","html_url":"https://github.com/openETCS/modeling/issues/629","id":99372665,"number":629,"title":"Radio Header (track to train): NID_LRBG lacks of NID_C information","user":{"login":"UweSteinkeFromSiemens","id":2225951,"avatar_url":"https://avatars.githubusercontent.com/u/2225951?v=3","gravatar_id":"","url":"https://api.github.com/users/UweSteinkeFromSiemens","html_url":"https://github.com/UweSteinkeFromSiemens","followers_url":"https://api.github.com/users/UweSteinkeFromSiemens/followers","following_url":"https://api.github.com/users/UweSteinkeFromSiemens/following{/other_user}","gists_url":"https://api.github.com/users/UweSteinkeFromSiemens/gists{/gist_id}","starred_url":"https://api.github.com/users/UweSteinkeFromSiemens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/UweSteinkeFromSiemens/subscriptions","organizations_url":"https://api.github.com/users/UweSteinkeFromSiemens/orgs","repos_url":"https://api.github.com/users/UweSteinkeFromSiemens/repos","events_url":"https://api.github.com/users/UweSteinkeFromSiemens/events{/privacy}","received_events_url":"https://api.github.com/users/UweSteinkeFromSiemens/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openETCS/modeling/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":{"login":"UweSteinkeFromSiemens","id":2225951,"avatar_url":"https://avatars.githubusercontent.com/u/2225951?v=3","gravatar_id":"","url":"https://api.github.com/users/UweSteinkeFromSiemens","html_url":"https://github.com/UweSteinkeFromSiemens","followers_url":"https://api.github.com/users/UweSteinkeFromSiemens/followers","following_url":"https://api.github.com/users/UweSteinkeFromSiemens/following{/other_user}","gists_url":"https://api.github.com/users/UweSteinkeFromSiemens/gists{/gist_id}","starred_url":"https://api.github.com/users/UweSteinkeFromSiemens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/UweSteinkeFromSiemens/subscriptions","organizations_url":"https://api.github.com/users/UweSteinkeFromSiemens/orgs","repos_url":"https://api.github.com/users/UweSteinkeFromSiemens/repos","events_url":"https://api.github.com/users/UweSteinkeFromSiemens/events{/privacy}","received_events_url":"https://api.github.com/users/UweSteinkeFromSiemens/received_events","type":"User","site_admin":false},"milestone":null,"comments":12,"created_at":"2015-08-06T07:07:54Z","updated_at":"2015-08-10T10:14:57Z","closed_at":null,"body":"The NID_LRBG field in the radio header Radio_Types_Pkg::Radio_TrackTrain_Header_T received from test environment lacks of NID_C information. \r\nNID_LRBG is an alternative representation of NID_C and NID_BG. NID_C must not be omitted, since there is no new country field provided as within packets (packet 5, ..).\r\n\r\nThe bug prevents CalculateTrainPosition from accepting linking information received via radio.\r\n\r\n@JakobGartner "},"comment":{"url":"https://api.github.com/repos/openETCS/modeling/issues/comments/129396483","html_url":"https://github.com/openETCS/modeling/issues/629#issuecomment-129396483","issue_url":"https://api.github.com/repos/openETCS/modeling/issues/629","id":129396483,"user":{"login":"UweSteinkeFromSiemens","id":2225951,"avatar_url":"https://avatars.githubusercontent.com/u/2225951?v=3","gravatar_id":"","url":"https://api.github.com/users/UweSteinkeFromSiemens","html_url":"https://github.com/UweSteinkeFromSiemens","followers_url":"https://api.github.com/users/UweSteinkeFromSiemens/followers","following_url":"https://api.github.com/users/UweSteinkeFromSiemens/following{/other_user}","gists_url":"https://api.github.com/users/UweSteinkeFromSiemens/gists{/gist_id}","starred_url":"https://api.github.com/users/UweSteinkeFromSiemens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/UweSteinkeFromSiemens/subscriptions","organizations_url":"https://api.github.com/users/UweSteinkeFromSiemens/orgs","repos_url":"https://api.github.com/users/UweSteinkeFromSiemens/repos","events_url":"https://api.github.com/users/UweSteinkeFromSiemens/events{/privacy}","received_events_url":"https://api.github.com/users/UweSteinkeFromSiemens/received_events","type":"User","site_admin":false},"created_at":"2015-08-10T10:14:57Z","updated_at":"2015-08-10T10:14:57Z","body":"I just picked the current model \"TestbenchIntegration.etp\" from github to continue with the analysis from Thursday and see, what has been changed meanwhile. \r\nI did not use TM_conversions::DECODE_NID_LRBG so far, so the misbehaviour should be similar to what I saw before. \r\n\r\nMy findings: \r\n- After passing BGs 353, 354, 351, the first datagram from radio with linking information is received. \r\n- NID_LRBG is 42600351: the intention, I suppose, is the equivalence of NID_C = 426, NID_BG = 351 as the reference LRBG. \r\n\r\nLet's calculate: \r\n- 42600351 d = 10 1000 1010 0000 0111 1001 1111 b. \r\n- The lower 14 bits are evaluated to NID_BG = 00 0111 1001 1111 b = 1951 d. \r\n- The upper 12 bits are evaluated to NID_C  =  10 1000 1010 00 b = 2600 d. \r\n- The upper 12 bits shortened to 10 bits (only 10 bits are foreseen for NID_C): \r\n1000 1010 00 b = 552 d.\r\n\r\nI think, this is not what was intended. I could propably compensate this as a dirty fix at the inputs of calculateTrainPosition, but this might not be the best place.\r\n"}}