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

docs: add Slack tutorial #285

Merged
merged 3 commits into from
Jan 17, 2025
Merged

docs: add Slack tutorial #285

merged 3 commits into from
Jan 17, 2025

Conversation

pilartomas
Copy link
Contributor

@pilartomas pilartomas commented Jan 17, 2025

Signed-off-by: Tomas Pilar [email protected]

Which issue(s) does this pull-request address?

Closes: #

Description

Checklist

  • I have read the contributor guide
  • I have signed off on my commit
  • Linting passes: yarn lint or yarn lint:fix
  • Formatting is applied: yarn format or yarn format:fix
  • Unit tests pass: yarn test:unit
  • E2E tests pass: yarn test:e2e
  • Tests are included
  • Documentation is changed or added
  • Commit messages and PR title follow conventional commits

@pilartomas pilartomas force-pushed the feat-slack-tutorial branch 3 times, most recently from 7c741c0 to 342d99b Compare January 17, 2025 13:35
@pilartomas pilartomas changed the title feat: add Slack tutorial docs: add Slack tutorial Jan 17, 2025
@pilartomas pilartomas marked this pull request as ready for review January 17, 2025 13:35
@pilartomas pilartomas requested a review from a team as a code owner January 17, 2025 13:35
Signed-off-by: Tomas Pilar <[email protected]>
// Create Bee agent
const agent = new BeeAgent({
// We're using phi4 LLM ran locally via Ollama
llm: new OllamaChatLLM({ modelId: "phi4" }),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

phi4 is not supposed to do call tooling

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, it does it better than llama3.1, we couldn't make it to do two tool calls.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rather stick with llama3.1 / granite and update the system prompt accordingly.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made some changes, so the tutorial should work with llama 3.1

Signed-off-by: Matous Havlena <[email protected]>
Signed-off-by: Matous Havlena <[email protected]>
@matoushavlena matoushavlena merged commit 686899f into main Jan 17, 2025
4 checks passed
@matoushavlena matoushavlena deleted the feat-slack-tutorial branch January 17, 2025 22:05
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.

3 participants