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

error when running rust example without docker #24

Open
xsa-dev opened this issue Sep 28, 2023 · 4 comments
Open

error when running rust example without docker #24

xsa-dev opened this issue Sep 28, 2023 · 4 comments

Comments

@xsa-dev
Copy link

xsa-dev commented Sep 28, 2023

I'm attempting to execute the examples/rust/top-ai-launches-hn example you graciously shared. Following the "cargo run" step, here's the outcome:

   Compiling top-ai-launches-hn v0.1.0 (/Users/xsa-osx/RustroverProjects/code-dagger)
error[E0432]: unresolved import `_chidori::create_change_value`
  --> src/main.rs:13:16
   |
13 | use _chidori::{create_change_value, NodeWillExecuteOnBranch};
   |                ^^^^^^^^^^^^^^^^^^^ no `create_change_value` in the root

warning: unused import: `std::env`
 --> src/main.rs:2:5
  |
2 | use std::env;
  |     ^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

warning: unused import: `std::net::ToSocketAddrs`
 --> src/main.rs:3:5
  |
3 | use std::net::ToSocketAddrs;
  |     ^^^^^^^^^^^^^^^^^^^^^^^

warning: unused imports: `Message`, `SmtpTransport`
 --> src/main.rs:6:14
  |
6 | use lettre::{Message, SmtpTransport, Transport};
  |              ^^^^^^^  ^^^^^^^^^^^^^

warning: unused import: `lettre::transport::smtp::authentication::Credentials`
 --> src/main.rs:7:5
  |
7 | use lettre::transport::smtp::authentication::Credentials;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `lettre::transport::smtp::Error`
 --> src/main.rs:8:5
  |
8 | use lettre::transport::smtp::Error;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `lettre::transport::smtp::response::Response`
 --> src/main.rs:9:5
  |
9 | use lettre::transport::smtp::response::Response;
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: unused import: `serde_json::json`
  --> src/main.rs:12:5
   |
12 | use serde_json::json;
   |     ^^^^^^^^^^^^^^^^

warning: unused imports: `DenoCodeNodeCreateOpts`, `serialized_value_to_string`
  --> src/main.rs:15:67
   |
15 | ...dori, CustomNodeCreateOpts, DenoCodeNodeCreateOpts, GraphBuilder, Handler, PromptNodeCreateOpts, serialized_value_to_string};
   |                                ^^^^^^^^^^^^^^^^^^^^^^                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^

warning: variable does not need to be mutable
  --> src/main.rs:37:21
   |
37 |                 let mut story: Story = client.get(&resource).send().await?.json().await?;
   |                     ----^^^^^
   |                     |
   |                     help: remove this `mut`
   |
   = note: `#[warn(unused_mut)]` on by default

warning: unused import: `Transport`
 --> src/main.rs:6:38
  |
6 | use lettre::{Message, SmtpTransport, Transport};
  |                                      ^^^^^^^^^

For more information about this error, try `rustc --explain E0432`.
warning: `top-ai-launches-hn` (bin "top-ai-launches-hn") generated 10 warnings
error: could not compile `top-ai-launches-hn` (bin "top-ai-launches-hn") due to previous error; 10 warnings emitted

Where might I be making a mistake?
Appreciate your help!

@hackintoshrao
Copy link
Contributor

Thanks for reporting the issue. This should fix it #27, would love to know your feedback and if you need any help building your agent using Chidori.

PS: I'm an user trying to build agents with Chidori, would love to know more thoughts from other members of the community.

@xsa-dev
Copy link
Author

xsa-dev commented Oct 10, 2023

I've tried it out, and I can see that the compilation errors have been fixed. However, the script still isn't functioning as expected. Would it be alright if I take a shot at debugging and improving the example? I'd like to submit a pull request with these improvements to save you the issue.

@hackintoshrao
Copy link
Contributor

Screenshot 2023-10-16 at 12 42 40 AM

It's getting stuck in the infinite loop after the execution of custom node, the prompt node is not getting called, I just started investigating it. Please feel free to share your observation if you managed to look into it.

@hackintoshrao
Copy link
Contributor

@xsa-dev This conversation provides more context, feel free to jump into the conversation in discord https://discord.com/channels/1132365768986742927/1132907110557024256/1163529198611222579

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

2 participants