Skip to content

Conversation

OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
Docs? no
Issues --
License MIT

@OskarStark OskarStark self-assigned this Aug 30, 2025
@OskarStark OskarStark requested a review from chr-hertel as a code owner August 30, 2025 09:57
@carsonbot carsonbot added Run examples Issues & PRs about the example scripts Feature New feature Status: Needs Review labels Aug 30, 2025
@OskarStark OskarStark merged commit 698d71b into symfony:main Aug 30, 2025
7 checks passed
@chr-hertel
Copy link
Member

There's no example for this - but let's fix that ...

chr-hertel added a commit that referenced this pull request Aug 31, 2025
This PR was merged into the main branch.

Discussion
----------

[Examples] Add example for RAG with ChromaDB

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Docs?         | no
| Issues        |
| License       | MIT

Follows #381

Commits
-------

e11aab1 Add example for RAG with ChromaDB
derrabus pushed a commit to derrabus/symfony-ai that referenced this pull request Sep 12, 2025
I started to get 400 Bad Request from Gemini API, not sure if it was due
to changing a model or what. Changing `system_instruction.parts` into a
list like in the
[docs](https://ai.google.dev/gemini-api/docs/text-generation#system-instructions)
fixed it.

Tested at least with `gemini-1.5-pro` and `gemini-2.5-pro` and
`gemini-2.0-flash`, works fine.
derrabus pushed a commit to derrabus/symfony-ai that referenced this pull request Sep 12, 2025
This PR was merged into the main branch.

Discussion
----------

fix: Gemini system prompt

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        |
| License       | MIT

Cherry picking php-llm/llm-chain#381

Commits
-------

88be4b3 fix: Gemini system prompt (symfony#381)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature Run examples Issues & PRs about the example scripts Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants