Skip to content

Long Duration Survey-In Greater Than 65535-seconds #117

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

Merged
merged 2 commits into from
Feb 28, 2022

Conversation

russ396
Copy link
Contributor

@russ396 russ396 commented Feb 27, 2022

The following changes allow for the library to perform a survey-in longer that 65535-seconds. Currently the observation time is uint16_t allowing for 65535-seconds. I wish to perform survey at 86400-seconds (24-hours). This requires a uint32_t to store the value.

Allow for survey-in observation times greater that 65535 (16-bit) seconds.  By changing to uint32_t will allow library to have observation time of 86400 seconds (24-hour) survey-in time.
Allow for uint32_t survey-in observation time to support >65535-seconds.  This will allow for 24-hour (86400-second) survey-in times.
@PaulZC
Copy link
Collaborator

PaulZC commented Feb 28, 2022

Hi @russ396 ,
Thank you for submitting this PR.
Unfortunately your changes break backwards-compatibility. I will need to add some additional overloads to correct that. But I will include your changes.
In future, please use the release_candidate branch. Please see CONTRIBUTING.md for details.
Best wishes,
Paul

@PaulZC
Copy link
Collaborator

PaulZC commented Feb 28, 2022

Hi @russ396 ,
I have included the changes you needed, but I had to add new functions to maintain backward-compatibility and avoid ambiguity errors. Please see the release notes for more details.
Best wishes,
Paul

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.

2 participants