Skip to content
This repository has been archived by the owner on May 29, 2022. It is now read-only.

Fixed Glonass MFtime in Rinex 2.11 store\read\write mismatch #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eugenyk
Copy link

@eugenyk eugenyk commented Sep 12, 2016

Suppose we have a rinex file

     2.11           GLONASS NAV DATA                        RINEX VERSION / TYPE
ASRINEXG V1.1.0 VM  AIUB                19-FEB-98 10:42     PGM / RUN BY / DATE 
STATION ZIMMERWALD                                          COMMENT             
                                                            END OF HEADER       
13 90  1  2  0  4  0.0 1.230000000000e-01 1.231100000000e+02 1.540000000000e+02 
    1.100000000000e+00 4.100000000000e+00 6.100000000000e+00 1.200000000000e+01 
    2.100000000000e+00 5.200000000000e+00 6.100000000000e+00 1.230000000000e+02 
    3.100000000000e+00 6.300000000000e+00 6.900000000000e+00 1.255500000000e+01 

and if we read it via Rinex3NavData then save it with Rinex3NavData when we get this rinex file

      2.11          NAVIGATION                              RINEX VERSION / TYPE
ASRINEXG V1.1.0 VM  AIUB                20150216 071100 UTC PGM / RUN BY / DATE
STATION ZIMMERWALD                                          COMMENT
                                                            END OF HEADER
13 90  1  2  0  4  0.0 1.230000000000e-01 1.231100000000e+02 1.729540000000e+05
    1.100000000000e+00 4.100000000000e+00 6.100000000000e+00 1.200000000000e+01
    2.100000000000e+00 5.200000000000e+00 6.100000000000e+00 1.230000000000e+02
    3.100000000000e+00 6.300000000000e+00 6.900000000000e+00 1.255500000000e+01

look at 1.540000000000e+02 and 1.729540000000e+05 values. It is the mismatch.

It happens because of https://github.com/SGL-UT/GPSTk/blob/master/core/lib/FileHandling/RINEX3/Rinex3NavData.cpp#L1204

we store it fixed, but when we are writing it https://github.com/SGL-UT/GPSTk/blob/master/core/lib/FileHandling/RINEX3/Rinex3NavData.cpp#L956
we don't make the reverse operation.

@masamitsutech
Copy link
Contributor

Before this pull request can be reviewed for the next stable release, please update the branch to resolve possible merge conflicts.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants