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

[VID] Migrate jf-vid into vid crate #2679

Open
mrain opened this issue Feb 25, 2025 · 0 comments
Open

[VID] Migrate jf-vid into vid crate #2679

mrain opened this issue Feb 25, 2025 · 0 comments

Comments

@mrain
Copy link
Contributor

mrain commented Feb 25, 2025

Move ADVZ implementation into vid crate.

sveitser pushed a commit that referenced this issue Mar 7, 2025
* Don't lock channels

* cargo fix

* Allow killing the connected network handler

* Fix issues

* Remove State from NetworkNodeHandle

* Remove async where it's unused

* fix async std build

* Fix erros a little

* Starting VID request response

* handle network event

* Bubble up request/response

* Make Request and Response just bytes at network layer

* Adding to ConnectedNetwork

* Hooks for request/response in handle + trait

* fix request

* Remove request and response tasks for now

* update mod.rs

* Hooked up response flow to ConnectedNetwork

* Refactor interface to return result to caller

* Add request and response to message struct

* Clean up some message stuff

* Fix build error

* Hook up request and response fully

* Review myself, impl functions for combine

* Change Receiver interface to spawn task

* try_send instead of send

* fix lint

* clean up request response event handle fn

* fix build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant