Skip to content

00 by encoding#136

Merged
huangminghuang merged 2 commits intoobjectcomputing:masterfrom
martinfantini:00_by_encodeing
Feb 26, 2025
Merged

00 by encoding#136
huangminghuang merged 2 commits intoobjectcomputing:masterfrom
martinfantini:00_by_encodeing

Conversation

@martinfantini
Copy link
Copy Markdown
Contributor

In the case of the encoder: If you use the active_message_id_, which data type, which is int64_t - for the template type Id -. It will generate an:

0x00 0xTemplateId

In the FAST stream, in this case it is necessary to send the Template Id without the 0x00 and this is done in the library with a unsigned type. That is why, you have to use the template_id to encode in the function.

@huangminghuang
Copy link
Copy Markdown
Contributor

Please rebase your work on top the latest main branch so that CI jobs can be successfully run.

@martinfantini
Copy link
Copy Markdown
Contributor Author

Please rebase your work on top the latest main branch so that CI jobs can be successfully run.

Done.

@huangminghuang
Copy link
Copy Markdown
Contributor

I made a mistake by merging a non-working PR133 before you rebase. I have already revert that PR at commit 1fecca6. Unfortunately, it requires you to rebase all your pending PRs to the current master head AGAIN; otherwise, the CI won't work. Sorry for the inconvenience.

@huangminghuang huangminghuang merged commit 3e00da0 into objectcomputing:master Feb 26, 2025
@martinfantini martinfantini deleted the 00_by_encodeing branch July 28, 2025 20:30
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