-
Notifications
You must be signed in to change notification settings - Fork 1.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
rt17.c code change #40
Comments
The raw->time is previously initialized by the current CPU time in RTKNAVI, RTKRCV, STRSVR or STR2STR. It is also initialized in RTKCONV or CONVBIN by user input with input dialog or -tr option. Other formats have the same problem (lack of week info, how to resolve week number) typically in RTCM. |
It was definitely zero when I went through the code in the debugger at which point it was set to a negative value by the referenced lines of code. It was CONVBIN running in the debugger. There must be some code path somewhere that doesn’t always initialize it. |
Would you use -tr option to specify approx. receiving time for CONVBIN? |
If you’re asking me what I would do, I would add the additional line of code I proposed earlier. :-) |
This issue is made moot by code changes included in my pull request #45 (RT17 updates) and can be closed if/when that pull request is merged. |
I see what you did there in 2.4.3 beta, in rt17.c/get_time() using utc2gpst(timeget()) instead of only timeget(). Thank you for that important code review and correction! I appreciate it. Could you please do the same sort of correction with the somewhat differing RT17 Update code I submitted? Thank you. |
They are merged to rtklib 2.4.2 p10. Thanks. |
Where can I get rtklib´s libraries in COFF format, or their source code? Thank you. |
Thank you very much for making certain needed code changes in rt17.c. I was hoping you would. :-)
However, the following code change which you made in rt17.c/get_time() causes the week to be set to a negative value when raw->time is zero. I suspect that is not what you intended to have happen with that particular change.
Background: Unlike everyone else, Trimble does not give us the gps week number. That's why raw->time is zero there. When we don't know what the gps week number is, we just have to do the best we can. I went to considerable lengths in rt17.c to determine the correct gps week number whenever possible. When all else fails and we still don't know the gps week number when we need it, most of the time the current gps week number based on the current gps time is the correct choice. When it is not, such as when processing an old file, the user can use the -WEEK=n receiver dependent option. Please note that UNAVCO TEQC handles this the same way.
The rt17.c/get_time() code change in question:
if 0
else
endif
I suggest instead perhaps something similar to the following code:
The text was updated successfully, but these errors were encountered: