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.

Sign up for GitHub

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

Jump to bottom

added tools support for fireworks.ai #4046

Open
wants to merge 57 commits into
base: main
Choose a base branch
from
Open

added tools support for fireworks.ai #4046

wants to merge 57 commits into from
+676 −881

Conversation

2hy6v
Copy link

@2hy6v 2hy6v commented Feb 8, 2025

Description

Added support for tools for the models who support tooling running via fireworks.ai.
Its also using openai api with some quirks.
Left behind explanation in comment section.

https://docs.fireworks.ai/guides/function-calling

Testing instructions

If one wanted to do so:

  1. Check if tooling still works with other providers (I have no other providers available to me, but the changes are just a few lines and I don't think anything can go wrong here)
  2. Use fireworks.ai as provider with and check if tooling works (it does for me now)

Todo

Proper versioning, idk how you guys handle this nor if I should have done anything in that regard since its my first contribution. I could not find anything regarding that in the Contributing.md

sestinj and others added 30 commits February 5, 2025 13:50
@sestinj
comment out toast
ee86344
@sestinj
remove unused workspace switcher
4106405
@sestinj
border-solid on scope select
fc23e41
@sestinj
remove AboutAssistantDialog
284ad97
@sestinj
open control plane links with orgSlug
c401e18
@sestinj
padding to top of assistant selector
19f0d6b
@sestinj
remove "welcome to continue for teams"
d3c9f67
@sestinj
adjust padding of assistant select
daa1c0a
@sestinj
color / style updates
cd5f4a7
@sestinj
fix timing bug in MCP loading
90a150d
@sestinj
always show tools button, but have disabled
26eb176
@sestinj
add tooltip for disabled tools
57b4c14
@sestinj
favicon for mcp
c3acd19
@sestinj
improvements to tool call display
35584f2
@sestinj
prioritize org over user secrets
dcf6c10
@RomneyDa
remove LocalAfterFreeTrial
c6eb626
@RomneyDa
local onboarding edits
4e38172
@sestinj
fix opt+enter when currentFile ctx provider not configured
16d7e0a
@sestinj
update config-yaml version
ecec00b
@sestinj
adjust colors
d0d9e74
@sestinj
focus border
d8dac11
@sestinj
get rid of lag from generating session title
85c82cd
@sestinj
transition border color
cacddc5
@sestinj
Merge branch 'main' into nate/polish-client
62c4daf
@RomneyDa
onboarding style changes
49cdf13
@sestinj
claude 35 haiku, not 3
62be9e6
@sestinj
if any docs used, add the docs context provider
a92368c
@RomneyDa
fix file provider submenu loads
a016eb4
@RomneyDa
tutorial card updates
27820f6
@sestinj
update styling to match personal taste
511abbd
@netlify Netlify
Copy link

netlify bot commented Feb 8, 2025 •
edited
Loading

✅ Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 0370e4f
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67ab2d03aeb41e000867b921

@tomasz-stefaniak
Copy link
Collaborator

@2hy6v could you pull the latest main? One of the tests if failing, main should contain the fix. Other than that, it looks good 👌

RomneyDa and others added 15 commits February 10, 2025 10:19
@RomneyDa
merge main
f494ed8
@RomneyDa
Merge branch 'main' of https://github.com/continuedev/continue into d…
ec0f8fd 
…allin/jetrains-ui-tweaks
@RomneyDa
docs indexing not dependent on disableIndexing
3c05e75
@RomneyDa
failed docs p1
201bfef
@RomneyDa
remember failed docs index attempts
0613435
@RomneyDa
failed docs in global context
6442115
@RomneyDa
skip docs indexing if already failed
a665b97
@sestinj
Merge pull request continuedev#4008 from continuedev/dallin/ollama-em…
4f4ea97 
…bedder

Ollama embedder and Onboarding Style fixes
@sestinj
Merge pull request continuedev#4023 from continuedev/dallin/file-fall…
0ad3ea6 
…backs

File Context Provider: handle on create/delete, fix fallback items
@sestinj
Merge pull request continuedev#4026 from continuedev/dallin/jetrains-…
21cfde2 
…ui-tweaks

Tutorial Card and UI Tweaks
@sestinj
Merge pull request continuedev#4077 from continuedev/dallin/dont-retr…
62126a7 
…y-docs

Don't retry failed docs with identical config
@sestinj
Merge pull request continuedev#4067 from siemens/fix/update-dependencies
a9fb8c4 
fix: bump node.js dependencies
@sestinj
Merge pull request continuedev#4062 from pczekaj/o3-mini-tools
732eb24 
Add tools support for o3-mini
@2hy6v
added tools support for fireworks.ai
d4943b3
@2hy6v
Merge remote-tracking branch 'origin/add-tools-support-for-fireworks-…
0370e4f 
…ai' into add-tools-support-for-fireworks-ai
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@2hy6v @tomasz-stefaniak @sestinj @RomneyDa @pczekaj @bufferoverflow