Skip to content
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

Microseconds Error in sp3 Reader #64

Closed
Shaylah-Maria opened this issue Mar 23, 2020 · 2 comments
Closed

Microseconds Error in sp3 Reader #64

Shaylah-Maria opened this issue Mar 23, 2020 · 2 comments

Comments

@Shaylah-Maria
Copy link

The time in the file is:
2018 9 23 0 2 37.63292462

Georinex is saving datetime as:
2018-9-23T00:02:37.063292 (incorrect)

instead of:
2018-9-23T00:02:37.63292 (correct)

See first data point entry:
Screen Shot 2020-03-23 at 1 20 30 PM

What is stored inside georinex.to_datetime(data.time[ii]) (first element of the saved dates in sp3 file). this should equal the first date point in file above:
image

To fix, must multiply what is saved in microseconds by 10.

-Shaylah Mutschler

@scivision
Copy link
Member

Thank you I seem to have overlooked this. I have flagged this so I can examine and fix. Thanks for your patience and kindly feedback.

@mauriciodev
Copy link
Contributor

mauriciodev commented Sep 30, 2021

It seems that line 145 of sp3.py should have "ln[23:29]" instead of "ln[23:28]"

    microsecond=int(ln[23:29]),

I have already made a pull request to the sp3 file today to address sp3-a data. Should I add this fix to that?

mauriciodev added a commit to mauriciodev/georinex that referenced this issue Sep 30, 2021
scivision added a commit that referenced this issue Nov 26, 2021
fixes #64
Co-authored-by: Mauricio de Paulo <[email protected]>
scivision added a commit that referenced this issue Nov 26, 2021
fixes #64

Co-authored-by: Mauricio de Paulo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants