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

pntpos: pass in the receiver index; and antpos fix #542

Closed

Conversation

ourairquality
Copy link

This is a prerequisite to applying the antenna delta for the pntpos solutions - need to know if the base or rover delta is to be applied. Also fixes an OOB in rtkpos.c.

Pass in the receiver index, as a 'base' argument, which is 0 for the rover and 1 for the base, to allow the respective options to be selected rather than just using the rover options even for the base. Start making use of this: in rescode() this is now used to select the respective SNR and SNR mask.

For postpos.c where a one based receiver number and zero base index were both be used, fix an instance in antpos() of indexing into option anttype[] with an one base index which would be OOB for a value of
2. Try to clean up this code in this regard.

Pass in the receiver index, as a 'base' argument, which is 0 for the
rover and 1 for the base, to allow the respective options to be
selected rather than just using the rover options even for the
base. Start making use of this: in rescode() this is now used to
select the respective SNR and SNR mask.

For postpos.c where a one based receiver number and zero base index
were both be used, fix an instance in antpos() of indexing into option
anttype[] with an one base index which would be OOB for a value of
2. Try to clean up this code in this regard.
@ourairquality
Copy link
Author

Moved to #533

@ourairquality ourairquality deleted the pntpos-base-rov branch December 9, 2024 01:45
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

Successfully merging this pull request may close these issues.

1 participant