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.

Sign up for GitHub

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

Jump to bottom

Update to the latest commit of CLP core. #66

Merged
merged 3 commits into from
Jul 5, 2024
Merged

Update to the latest commit of CLP core. #66

merged 3 commits into from
Jul 5, 2024

Conversation

LinZhihao-723
Copy link
Member

References

Description

This PR updates to the latest clp commit. It contains the following changes:

  1. Update to the latest IR deserialization procedure. Tags will be deserialized first before deserializing the actual packet. Notice that UTC offset deserialization is not yet supported. It will be added in the future PR.
  2. Update the include path, so that src files include <clp/header.hpp> instead of <clp/components/core/src/clp/header.hpp>

Validation performed

Workflow passed (lint, build, and unit testing).

@LinZhihao-723 LinZhihao-723 requested a review from davidlion July 5, 2024 16:52
@davidlion davidlion changed the title Update to the latest clp submodule commit. Update to the latest commit of CLP core. Jul 5, 2024
@davidlion davidlion merged commit 5fe0b1d into main Jul 5, 2024
120 checks passed
@LinZhihao-723 LinZhihao-723 deleted the clp_update branch July 25, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidlion davidlion davidlion approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LinZhihao-723 @davidlion