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.

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

Fix the scout termination issue #182

Closed

Conversation

YuanYuYuan
Copy link
Contributor

@YuanYuYuan YuanYuYuan commented Apr 17, 2024

Closes #180.

@YuanYuYuan YuanYuYuan changed the title Fix/scout termination Fix the scout termination issue Apr 17, 2024
@YuanYuYuan YuanYuYuan changed the base branch from fix/uhlc to main April 17, 2024 14:59
@YuanYuYuan
Copy link
Contributor Author

Finally, we made it. We can supersede #173 with this PR as it has already included it. Once eclipse-zenoh/zenoh#946 is merged, we can switch the upstream back to eclipse-zenoh.

@Mallets
Copy link
Member

Mallets commented Apr 18, 2024

@wyfo can you please review?

@wyfo
Copy link
Contributor

wyfo commented Apr 18, 2024

The solution here has not to be modifying zenoh code. The issue is a deadlock in _Scout drop, so we just need to update this operation on zenoh-python side, I will push a PR.

@YuanYuYuan
Copy link
Contributor Author

Superseded by #188

@YuanYuYuan YuanYuYuan closed this Apr 22, 2024
@YuanYuYuan YuanYuYuan deleted the fix/scout-termination branch April 22, 2024 10:34
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.

[Bug] z_scout fails to terminate
4 participants