Skip to content

Add getTypeIndex and getPacketByIndex functions to Decoder#8

Merged
awann-4ai merged 23 commits into
Fastcode:mainfrom
awann-4ai:wann/expose_index_file
Feb 20, 2024
Merged

Add getTypeIndex and getPacketByIndex functions to Decoder#8
awann-4ai merged 23 commits into
Fastcode:mainfrom
awann-4ai:wann/expose_index_file

Conversation

@awann-4ai
Copy link
Copy Markdown
Collaborator

@awann-4ai awann-4ai commented Feb 12, 2024

Adds functions to allow the index timestamps to be requested from the Decoder and request packets using an index.

Copy link
Copy Markdown
Member

@TrentHouliston TrentHouliston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Partial review, will look more closely later

Comment thread nbsdecoder.d.ts Outdated
Comment thread src/Decoder.cpp Outdated
Comment thread src/Decoder.cpp Outdated
Comment thread src/Decoder.cpp Outdated
Comment thread nbsdecoder.d.ts Outdated
Comment thread nbsdecoder.d.ts Outdated
Comment thread nbsdecoder.d.ts Outdated
Comment thread nbsdecoder.d.ts Outdated
Comment thread src/Decoder.cpp Outdated
Comment thread tests/test_decoder.js Outdated
Comment thread tests/test_decoder.js Outdated
Comment thread tests/test_decoder.js Outdated
Comment thread tests/test_decoder.js Outdated
Comment thread tests/test_decoder.js Outdated
Copy link
Copy Markdown
Collaborator

@JosephusPaye JosephusPaye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor thing, otherwise JS and tests look good to me.

Should also update PR title about index property.

Comment thread src/Index.hpp Outdated
@awann-4ai awann-4ai changed the title Add index property and getPacketByIndex to Decoder Add getTypeIndex and getPacketByIndex functions to Decoder Feb 19, 2024
Copy link
Copy Markdown
Collaborator

@JosephusPaye JosephusPaye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@awann-4ai awann-4ai merged commit a970771 into Fastcode:main Feb 20, 2024
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.

3 participants