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

Receiving multiple Input from Web portal. #616

Open
XiaoKuge opened this issue Aug 8, 2024 · 11 comments
Open

Receiving multiple Input from Web portal. #616

XiaoKuge opened this issue Aug 8, 2024 · 11 comments

Comments

@XiaoKuge
Copy link

XiaoKuge commented Aug 8, 2024

Is your feature request related to a problem? Please describe.
We are developing a Web frontend that drives dora-flows at the backend. It seems that we can only setup a dynamic node at the begining of the dora flow to get input. The middle nodes won't be able to take the additional input.

Describe the solution you'd like
Provide the middle-nodes capability to receive input from frontend.

Describe alternatives you've considered
N/A

Additional context

Thank you!

from MAE team

@haixuanTao
Copy link
Collaborator

I'm not sure, I understand. Could you provide a simple example ?

@haixuanTao
Copy link
Collaborator

I think that you might have been blocked by: #606 not being merged.

Now that it's merged it should be easier to have receive-only dynamic node

@chengzi0103
Copy link

I think that you might have been blocked by: #606 not being merged.

Now that it's merged it should be easier to have receive-only dynamic node

How should I use this configuration? Could you provide an example of how to configure synchronized_start in YAML or Python?

@haixuanTao
Copy link
Collaborator

Do you need that all nodes are ready to start or do you want them to start independently from your web front end?

@chengzi0103
Copy link

Do you need that all nodes are ready to start or do you want them to start independently from your web front end?

I want them to start independently from the web front end.

@haixuanTao
Copy link
Collaborator

Yes, so you don't need to configure anything the new merged PR will allow you to do just that.

@chengzi0103
Copy link

Yes, so you don't need to configure anything the new merged PR will allow you to do just that.

However, have you tried including multiple dynamic nodes in a dataflow? When I started one of the nodes, there was no response at all.

Can you take a look at this issue?

#607

@haixuanTao
Copy link
Collaborator

haixuanTao commented Aug 9, 2024

Yes, with the latest version that I have just merged. You can include multiple dynamic nodes. You can even include them in the same python script.

@chengzi0103
Copy link

Yes, with the latest version that I have just merged. You can include multiple dynamic nodes. You can even include them in the same python script.

How should I get your updates? Should I reinstall Dora-RS? If so, which version should I install?

@haixuanTao
Copy link
Collaborator

Yes so, to update to latest main:

sudo rm $(which dora)
pip uninstall dora-rs

## dora binary
git clone https://github.com/dora-rs/dora.git
cd dora
cargo build --release -p dora-cli
PATH=$PATH:$(pwd)/target/release

## Python API
maturin develop -m apis/python/node/Cargo.toml

dora --help

@chengzi0103
Copy link

Yes so, to update to latest main:

sudo rm $(which dora)
pip uninstall dora-rs

## dora binary
git clone https://github.com/dora-rs/dora.git
cd dora
cargo build --release -p dora-cli
PATH=$PATH:$(pwd)/target/release

## Python API
maturin develop -m apis/python/node/Cargo.toml

dora --help

Alright, thank you.

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

No branches or pull requests

3 participants