Commit a7d8972
committed
feature #837 add support for the albert platform configuration (jvancoillie)
This PR was merged into the main branch.
Discussion
----------
add support for the albert platform configuration
| Q | A
| ------------- | ---
| Bug fix? |no
| New feature? | yes <!-- please update src/**/CHANGELOG.md files -->
| Docs? | no <!-- required for new features -->
| Issues | Fix #836
| License | MIT
This PR introduces support for configuring the **Albert** platform under the `ai.platforms` section, following the pattern of existing platforms (OpenAI, Mistral, Ollama, etc.).
The underlying Albert integration was already implemented, but the `albert` platform key was not recognized during configuration parsing.
```yaml
ai:
platforms:
albert:
api_key: '%env(ALBERT_API_KEY)%'
base_url: '%env(ALBERT_API_URL)%'
```
The configuration uses `base_url` because this is the parameter expected by the Albert platform factory.
Some platforms use `host` instead, which might cause confusion. If needed, I'm open to aligning naming conventions in a follow-up discussion.
I did not define a default `base_url`, but we could consider providing the known Albert endpoint (`https://albert.api.etalab.gouv.fr/v1`) if that aligns with the project’s policy on defaults.
While working on this, I also noticed that the list of Albert models may not be fully aligned with the current API. I can open a separate issue to track that, unless this work is already in progress.
Commits
-------
0e846d2 feat(aibundle): integrate Albert platform supportFile tree
5 files changed
+40
-1
lines changed- src
- ai-bundle
- config
- src
- tests/DependencyInjection
- platform/src/Bridge/Albert
5 files changed
+40
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
32 | 42 | | |
33 | 43 | | |
34 | 44 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
28 | 29 | | |
29 | 30 | | |
30 | 31 | | |
| |||
88 | 89 | | |
89 | 90 | | |
90 | 91 | | |
| 92 | + | |
91 | 93 | | |
92 | 94 | | |
93 | 95 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
| 46 | + | |
46 | 47 | | |
47 | 48 | | |
48 | 49 | | |
| |||
292 | 293 | | |
293 | 294 | | |
294 | 295 | | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
295 | 316 | | |
296 | 317 | | |
297 | 318 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2817 | 2817 | | |
2818 | 2818 | | |
2819 | 2819 | | |
| 2820 | + | |
| 2821 | + | |
| 2822 | + | |
| 2823 | + | |
2820 | 2824 | | |
2821 | 2825 | | |
2822 | 2826 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
| 19 | + | |
19 | 20 | | |
20 | 21 | | |
21 | 22 | | |
| |||
29 | 30 | | |
30 | 31 | | |
31 | 32 | | |
| 33 | + | |
32 | 34 | | |
33 | 35 | | |
34 | 36 | | |
| |||
52 | 54 | | |
53 | 55 | | |
54 | 56 | | |
55 | | - | |
| 57 | + | |
56 | 58 | | |
57 | 59 | | |
58 | 60 | | |
| |||
0 commit comments