gitea/astrololo
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(llm): konfiguracja per dostawca — przelacznik w UI byl iluzja #15

Open
gitea wants to merge 1 commits from fix/llm-per-provider-config into master
pull from: fix/llm-per-provider-config
merge into: gitea:master
Conversation 0 Commits 1 Files Changed 2
+107 -12
gitea commented 2026-07-21 20:32:23 +00:00
Owner
Copy Link
Copy Source

Blad

UI pozwala wybrac dostawce przy kazdym zadaniu, ale factory.py czytalo jedna wspolna
trojke LLM_MODEL / LLM_BASE_URL / LLM_API_KEY dla wszystkich dostawcow.

Na klastrze LLM_BASE_URL trzeba ustawic na lokalny model (domyslne localhost:11434
w podzie nie istnieje) — i wtedy:

Wybor w UI Co sie naprawde dzialo
OpenAI zadanie szlo do Ollamy
Anthropic proszono Anthropic o model llama
— jeden klucz nie moze byc kluczem OpenAI i Anthropic naraz

Czyli nie bylo miejsca, w ktore dalo sie sensownie wpisac klucze do chmury — stad to PR,
zanim odpowiem na pytanie „gdzie dac tokeny".

Naprawa

Konfiguracja per dostawca:

LOCAL_MODEL, LOCAL_BASE_URL
OPENAI_MODEL, OPENAI_BASE_URL, OPENAI_API_KEY
ANTHROPIC_MODEL, ANTHROPIC_BASE_URL, ANTHROPIC_API_KEY

Zgodnosc wstecz zachowana: wspolne LLM_* dziala nadal, ale stosuje sie wylacznie do
dostawcy domyslnego (LLM_PROVIDER). Instalacja jednodostawcowa nie wymaga zmian, a
pozostali dostawcy nie dziedzicza cudzej konfiguracji.

Przy okazji: Anthropic dostal brakujaca walidacje klucza (mial ja tylko OpenAI, wiec bez
klucza szedl request i dopiero API zwracalo blad), a komunikat wskazuje teraz konkretna
zmienna do ustawienia zamiast ogolnego LLM_API_KEY.

Testy

Nowe testy regresyjne pilnuja, ze ustawienia jednego dostawcy nie przeciekaja na
pozostalych — w tym dokladny scenariusz z bledu (skonfigurowany lokalny + wybor OpenAI).
Calosc: 143 passed / 1 skipped.

## Blad UI pozwala wybrac dostawce przy **kazdym zadaniu**, ale `factory.py` czytalo **jedna wspolna** trojke `LLM_MODEL` / `LLM_BASE_URL` / `LLM_API_KEY` dla wszystkich dostawcow. Na klastrze `LLM_BASE_URL` **trzeba** ustawic na lokalny model (domyslne `localhost:11434` w podzie nie istnieje) — i wtedy: | Wybor w UI | Co sie naprawde dzialo | |---|---| | OpenAI | zadanie szlo **do Ollamy** | | Anthropic | proszono Anthropic o **model llama** | | — | jeden klucz nie moze byc kluczem OpenAI **i** Anthropic naraz | Czyli **nie bylo miejsca, w ktore dalo sie sensownie wpisac klucze do chmury** — stad to PR, zanim odpowiem na pytanie „gdzie dac tokeny". ## Naprawa Konfiguracja **per dostawca**: ``` LOCAL_MODEL, LOCAL_BASE_URL OPENAI_MODEL, OPENAI_BASE_URL, OPENAI_API_KEY ANTHROPIC_MODEL, ANTHROPIC_BASE_URL, ANTHROPIC_API_KEY ``` **Zgodnosc wstecz zachowana**: wspolne `LLM_*` dziala nadal, ale stosuje sie **wylacznie do dostawcy domyslnego** (`LLM_PROVIDER`). Instalacja jednodostawcowa nie wymaga zmian, a pozostali dostawcy nie dziedzicza cudzej konfiguracji. Przy okazji: **Anthropic dostal brakujaca walidacje klucza** (mial ja tylko OpenAI, wiec bez klucza szedl request i dopiero API zwracalo blad), a komunikat wskazuje teraz konkretna zmienna do ustawienia zamiast ogolnego `LLM_API_KEY`. ## Testy Nowe testy regresyjne pilnuja, ze ustawienia jednego dostawcy **nie przeciekaja** na pozostalych — w tym dokladny scenariusz z bledu (skonfigurowany lokalny + wybor OpenAI). Calosc: **143 passed / 1 skipped**.
gitea added 1 commit 2026-07-21 20:32:24 +00:00
fix(llm): konfiguracja per dostawca — przelacznik w UI byl iluzja
Testy / Testy warstwy logicznej (silnik) (push) Successful in 10m29s
Details
Testy / Testy warstwy prezentacji (dostęp do baz) (push) Successful in 9m53s
Details
Testy / Build obrazu silnika B (swisseph) (push) Failing after 25s
Details
Testy / Kontrola składni wszystkich warstw (push) Successful in 21s
Details
Testy / Testy warstwy logicznej (silnik) (pull_request) Successful in 10m49s
Details
Testy / Testy warstwy prezentacji (dostęp do baz) (pull_request) Successful in 9m55s
Details
Testy / Build obrazu silnika B (swisseph) (pull_request) Failing after 28s
Details
Testy / Kontrola składni wszystkich warstw (pull_request) Successful in 23s
Details
5203ba9e76 
UI pozwala wybrac dostawce przy KAZDYM zadaniu, ale factory czytalo jedna
wspolna trojke LLM_MODEL / LLM_BASE_URL / LLM_API_KEY dla wszystkich. Na
klastrze LLM_BASE_URL trzeba ustawic na lokalny model (localhost:11434 w podzie
nie istnieje) — i wtedy:
  - wybor „OpenAI" wysylal zadanie do Ollamy,
  - LLM_MODEL=llama3.1:8b kazal Anthropic uzyc modelu llama,
  - jeden LLM_API_KEY nie moze byc kluczem OpenAI i Anthropic naraz.
Czyli nie bylo miejsca, w ktore dalo sie sensownie wpisac klucze do chmury.

- konfiguracja per dostawca: <DOSTAWCA>_MODEL / _BASE_URL / _API_KEY
  (LOCAL_*, OPENAI_*, ANTHROPIC_*),
- zgodnosc wstecz: wspolne LLM_* dziala nadal, ale stosuje sie WYLACZNIE do
  dostawcy domyslnego (LLM_PROVIDER) — instalacja jednodostawcowa bez zmian,
- Anthropic dostal brakujaca walidacje klucza (mial ja tylko OpenAI),
- komunikat bledu wskazuje konkretna zmienna do ustawienia.

Testy regresyjne pilnuja, ze ustawienia jednego dostawcy NIE przeciekaja na
pozostalych. Calosc: 143 passed / 1 skipped.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Some checks are pending
Testy / Testy warstwy logicznej (silnik) (push) Successful in 10m29s
Details
Testy / Testy warstwy prezentacji (dostęp do baz) (push) Successful in 9m53s
Details
Testy / Build obrazu silnika B (swisseph) (push) Failing after 25s
Details
Testy / Kontrola składni wszystkich warstw (push) Successful in 21s
Details
Testy / Testy warstwy logicznej (silnik) (pull_request) Successful in 10m49s
Details
Testy / Testy warstwy prezentacji (dostęp do baz) (pull_request) Successful in 9m55s
Details
Testy / Build obrazu silnika B (swisseph) (pull_request) Failing after 28s
Details
Testy / Kontrola składni wszystkich warstw (pull_request) Successful in 23s
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/llm-per-provider-config:fix/llm-per-provider-config
git checkout fix/llm-per-provider-config
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
gitea
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: gitea/astrololo#15
Delete Branch "fix/llm-per-provider-config"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?