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

Proposal: Make lli and ssi optioal #20

Closed
VOMAY opened this issue Jul 26, 2018 · 1 comment
Closed

Proposal: Make lli and ssi optioal #20

VOMAY opened this issue Jul 26, 2018 · 1 comment

Comments

@VOMAY
Copy link
Contributor

VOMAY commented Jul 26, 2018

Dear Scivision,

Your Rinex3 parser does not only read all observations, but also all lli and ssi indicators. Both come with every phase observation and are stored as floats. They take up a large share of the output xarray size.

From my experience, those indicators are irrelevant for most applications.

I propose to make the lli and ssi parsing optional to save memory and potentially also parsing time. Besides, parsing/storing them as integers would make the xarray output more consistent with the nature/definition of the indicators. Personally, I would ignore thr indicators completely.

Thanks for your hard work!

Best Regards
Volker

@scivision
Copy link
Member

scivision commented Jul 26, 2018

Yes this is now a feature--by default LLI and SSI are off. This provides a factor of 2 speedup at least, and ~ 30% memory savings
-useindicators turns LLI and SSI on again

https://github.com/scivision/georinex/releases/tag/v1.5.1

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

2 participants