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

Wrap zenoh session with a shared pointer #332

Closed
wants to merge 149 commits into from
Closed

Wrap zenoh session with a shared pointer #332

wants to merge 149 commits into from

Conversation

YuanYuYuan
Copy link
Contributor

Mallets and others added 30 commits August 9, 2024 14:32
@Mallets @YuanYuYuan
Update style.yaml
e2fabb6 
Signed-off-by: Luca Cominardi <[email protected]>
@Mallets @YuanYuYuan
Update check_logging.yaml
6cd6dbe 
Signed-off-by: Luca Cominardi <[email protected]>
@Mallets @YuanYuYuan
Update build.yaml
dcefb68 
Signed-off-by: Luca Cominardi <[email protected]>
@YuanYuYuan
chore: configure the compiliation
14fe58c 
chore: checkout dev/1.0.0
@YuanYuYuan
chore: complete the 1st version
301a8ce
@YuanYuYuan
fix: memory leak
8ef92b5
@YuanYuYuan
wip
06b6fa4
@YuanYuYuan
fix: z_error_t -> z_result_t
71e7c5b
@fuzzypixelz @YuanYuYuan
Fix scouting/*/autoconnect/* per eclipse-zenoh/zenoh@b31a410 (#3)
a717270
@YuanYuYuan
chore: checkout the local zenoh-c
cfa63d9
@YuanYuYuan
chore: polish z_open
12dc820
@YuanYuYuan
fix: segfault
d331b74
@YuanYuYuan
feat: z_bytes_serialize_from_slice without copy
1db4bef
@YuanYuYuan
chore: switch back to https://github.com/eclipse-zenoh/zenoh-c
d12f189
@fuzzypixelz @YuanYuYuan
Initialize query_ member of ZenohQuery
909329b
@YuanYuYuan
refactor: use z_owned_slice_t instead
511c6f6
@YuanYuYuan
chore: adapt the latest change of zenoh-c dev/1.0.0
044d25b
@YuanYuYuan
chore: use strncmp to avoid copying
d79cf5e
@YuanYuYuan
refactor: use z_view_keyexpr_t to avoid copying
6637400
@YuanYuYuan
feat: implment the SHM feature
83334c8
@YuanYuYuan
chore: adpat the new changes from zenoh-c and fix the bug in liveliness
bf0da8a
@YuanYuYuan
fix: segmentation fault due to the unallocated query memory
dfc95ad
@YuanYuYuan
fix: workaround the ZID parsing issue
cd071e5
@yellowhatter @YuanYuYuan
fix Zenoh Config read\check
4f842fd
@yellowhatter @YuanYuYuan
adopt to recent zenoh-c API changes
af2e344
@YuanYuYuan
chore: sync up with the rolling branch
0225a59
@imstevenpmwork
chore: update to self-hosted runners
36140fd
@YuanYuYuan
fix: adapt the latest change of batching config
080eb15
@YuanYuYuan
Merge remote-tracking branch 'origin/rolling' into dev/1.0.0
f60c2a2
@YuanYuYuan
build: deprecate the zenohc_debug and incldue the zenohc dependency i…
83a26f2 
…n the zenoh_c_vendor
YuanYuYuan and others added 27 commits November 20, 2024 01:12
@YuanYuYuan
chore: remove .gitignore
09798c3
@YuanYuYuan
fix: correct typo attachement to attachment
b6df66c
@YuanYuYuan
Merge branch 'rolling' into wip/merge-rolling
59203df
@YuanYuYuan
style: ament_cpplint and ament_uncrustify
9f78ca4
@YuanYuYuan
refactor: remove the warning of subscriber reliability QoS
7450bb2
@YuanYuYuan
fix: add back atexit handler
5ade015
@YuanYuYuan
merge: integrate rolling into dev/1.0.0
1e705c1
@YuanYuYuan
Revert "Don't shutdown contained entities. (ros2#313)"
2e3ad43 
This reverts commit 5366957.
@YuanYuYuan
Reapply "Don't shutdown contained entities. (ros2#313)"
c41ee5a 
This reverts commit 2e3ad43.
@YuanYuYuan
chore: include the string_array.h
aa19545
@YuanYuYuan
fix: include "rcutils/types/string_array.h"
b485a93
@fuzzypixelz
Fix z_view_string_t to std::string conversion (#45)
0dffa49 
* Fix `z_view_string_t` to `std::string` conversion

* Fix formatting
@YuanYuYuan
fix: address the comments
91df63c
@YuanYuYuan
style: ament_cpplint
70990e7
@YuanYuYuan
refactor: remove the workaround for the atexit shutdown
33f0bec
@YuanYuYuan
fix: correct the call by ref
b8ce65c
@YuanYuYuan
refactor: revert the header clean
435186a
@YuanYuYuan
refactor: zc_liveliness_* -> z_liveliness_* and bump up zenoh-c version
93afc16
@YuanYuYuan
refactor: reorder the cancel functions
17865d2
@YuanYuYuan
chore: reorder some lines of code
0255c37
@YuanYuYuan
refactor: add session_is_valid check
09b45c3
@YuanYuYuan
fixup! refactor: reorder the cancel functions
0e1cca1
@YuanYuYuan
fixup! refactor: zc_liveliness_* -> z_liveliness_* and bump up zenoh-…
09ecde7 
…c version
@YuanYuYuan
fixup! fix: address the comments
8f4949c
@YuanYuYuan
chore: address the comment
84d1267
@YuanYuYuan
refactor: wrap the zenoh session with a shared pointer
afd1a1e
@YuanYuYuan
refactor!: unify the initialized_ flag
1ecdd5c
@clalancette
Copy link
Collaborator

@YuanYuYuan This looks like it contains all of the changes from dev/1.0.0 plus this branch. Can you try doing a merge of rolling into this branch?

@YuanYuYuan
Copy link
Contributor Author

Superseded by #333

@YuanYuYuan YuanYuYuan closed this Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Issues caused by not shutting down the contained entity
10 participants
@YuanYuYuan @clalancette @Mallets @fuzzypixelz @yellowhatter @imstevenpmwork @evshary @JEnoch @Yadunund @gabrik