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

dora_next_event()'s delay is too long #609

Open
lassloss opened this issue Aug 5, 2024 · 3 comments
Open

dora_next_event()'s delay is too long #609

lassloss opened this issue Aug 5, 2024 · 3 comments
Labels
bug Something isn't working cli CLI

Comments

@lassloss
Copy link

lassloss commented Aug 5, 2024

Here are some examples of successes and failures during my testing:

  1. C++version, transfer 1.3 MB images, use current compiled target/debug/dora to run the project, and the delay in obtaining images is about 100ms when dora_next_event() is used. If using the directly downloaded compiled dora(v0.3.5) to run XXX.yml, the delay is 4ms (only for versions before July 19th).
  2. In versions after July 19th, when running the official example using downloaded compiled dora, the second node cannot receive the data from the previous node due to deserialization issues. This may be caused by the mismatch between the cargo install dora cli compiled dora and the latest C++library functions. Simultaneously using current compiled target/debug/dora to run the program successfully, but the latency is still around 100ms.
@github-actions github-actions bot added bug Something isn't working cli CLI labels Aug 5, 2024
@haixuanTao
Copy link
Collaborator

So, you have to use the release version if you want to have the best latency.

Make sure to use --release when compiling dora.

If you're using the latest main branch to compile C++, you should also use dora main branch to start your dataflow.

Otherwise, use https://github.com/dora-rs/dora/tree/v0.3.5 which is tagged to 0.3.5

@lassloss
Copy link
Author

lassloss commented Aug 6, 2024

If you're using the latest main branch to compile C++, you should also use dora main branch to start your dataflow.

When I use the latest main branch to compile C++, the latency is 100ms. How can I reduce the latency? Should I use --release? I'm not familar with cargo and haven't found where the dora is compiled. Thanks!

@Hennzau
Copy link
Collaborator

Hennzau commented Aug 6, 2024

Hi! What you should do is:

  • uninstall previous version of Dora: cargo uninstall dora-cli
  • install Dora-cli latest main (it will be in release mode): clone the Dora directory and launch cargo install --path binaries/cli inside of it
  • build the cxx node api release mode: inside the repository, launch cargo build --package dora-node-api-cxx --release and just follow the steps that are in the README of this subfolder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cli CLI
Projects
None yet
Development

No branches or pull requests

3 participants