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

Warteschlange? #2

Open
dpf-dd opened this issue Jul 24, 2024 · 15 comments
Open

Warteschlange? #2

dpf-dd opened this issue Jul 24, 2024 · 15 comments

Comments

@dpf-dd
Copy link

dpf-dd commented Jul 24, 2024

Zunächst vorweg: Sehr coole Idee für ein AddOn, das hat Zukunft :)

Ich habe soeben mal den aktuellen Stand ausprobiert, leider ohne Erfolg. Ich vermute, dass ich erstmal auf die Review-Funktion warten bevor es wieder funktioniert?

Currently, the AI-generated meta descriptions are saved in the task table. This change was made to avoid incorrect answers being written directly to the article. A review function will be implemented to check the generated meta descriptions before saving them to the article.

Ich bleibe hier mal interessiert dabei und teste gerne weiter :)

@omphteliba
Copy link
Owner

Hej Stefan, was geht denn nicht. Der Kommentar aus der README Seite, den Du zitiert hast, beschreibt eigentlich nur die Funktion. Das ist alles fertig.

@dpf-dd
Copy link
Author

dpf-dd commented Jul 24, 2024

Also ich habe bei meinem Test ein MetaInfo-Feld "med_caption" angelegt und ein Testbild in den Medienpool gelegt (ohne Eingabe von "med_caption"). API-Key ist vorhanden und gespeichert, bei Einstellungen habe ich unter "DB Feld für Image Description" mein MetaInfo-Feld "med_caption" hinterlegt. Dann wechsle ich in den Reiter "Bildbeschreibung", stelle bei Bild auf "einzeln" und wähle mit dem Mediabutton-Widget mein Testfoto aus dem Medienpool und klicke dann auf den grünen Button "In Warteschlage einstellen" (man beachte den Typo, da fehlt ein "n" ^^).

Resultat: Nichts. Am eingestellten Bild bleibt das Feld "med_caption" leer und ich habe auch nirgendwo eine Möglichkeit die Warteschlange oder deren Status einzusehen...

EDIT: Ich habe die Version aus dem Redaxo-Installer geladen und NICHT den GitHub-Stand hier...

@omphteliba
Copy link
Owner

So sieht die Warteschlange aus:
image

Kannst Du nachschauen, ob die Datenbank-Tabelle: rex_ff_gpt_tools_tasks angelegt wurde? Darin wird die Warteschlange verwaltet.

@dpf-dd
Copy link
Author

dpf-dd commented Jul 24, 2024

Die Tabelle ist angelegt und vorhanden, bleibt aber laut Adminer leer wenn ich wie oben beschrieben die API über mein Testbild laufen lasssen möchte... Ich klicke den grünen Button "In Warteschlange einstellen" und der Bereich "Aufgaben" fehlt bei mir komplett...

EDIT: Fehler gefunden. Ich muss unter "Sprache" den Eintrag DEUTSCH auch auswählen. Das war nicht wirklich offensichtlich, weil da voreingestellt schon "deutsch" drin steht. Das könnte man vielleicht intuitiver lösen?

Neues Problem: Wenn ich den Task ausführe passiert nichts. Es sagt zwar "erledigt" - aber das mein Feld "med_caption" ist immernoch leer.

Bildschirmfoto 2024-07-24 um 12 49 09

Ichsehe, dass du für das Abarbeiten der Tasks einen Cronjob mitlieferst. Ich nutze das Cronjob AddOn aber nicht. Fehlt da evtl. etwas was ich beim Hoster noch einstellen muss, damit die Cronjobs auch funktionieren? Da war doch irgendwas (dunkel die Erinnerung ist)
Bildschirmfoto 2024-07-24 um 12 55 53

@omphteliba
Copy link
Owner

Ah, ja, das kann gut sein, dass das ganze ohne Cronjob Addon nicht geht. Sollte ich in die Requirements reinschreiben.

@dpf-dd
Copy link
Author

dpf-dd commented Jul 24, 2024

Ich trenne Redaxo und Cronjobs grundsätzlich und verwalte Cronjobs ggf. direkt beim Hoster ein. Ich würde mir also wünschen, dass das auch ohne funktioniert... :)

Gibt es denn einen Grund, dass das Cronjob-AddOn zwingend benötigt wird (mir fällt keiner ein ^^)?

@omphteliba
Copy link
Owner

Eigentlich nicht: ich lasse vieles als Cronjob laufen, darum habe ich das so angelegt. Und der Knopf "Tasks ausführen" startet auch nur den Cronjob. Aber lässt sich ändern.

@dpf-dd
Copy link
Author

dpf-dd commented Jul 24, 2024

Würde ich gut finden. Es gilt auch zu bedenken, dass die günstigen Webpakete bei Hostern oft keine Cronjobs enthalten. Ich könnte dein AddOn z.B. mit einem all-inkl- tarif "privat" (Fünfer pro Monat) aufgrund der fehlenden Cronjobs nicht benutzen.

Abgeleitete Vermutung: Die Anzahl potenzieller Tester/Nutzer deines AddOns wird größer ohne Cronjob-Abhängigkeit :D

@omphteliba
Copy link
Owner

So @dpf-dd schau mal, ob die neueste Version jetzt für Dich geht. Ich habe das ganze so umgebaut, dass die Ausführung der Tasks nicht mehr im Cronjob stattfindet.

@Markus-GS
Copy link

Markus-GS commented Sep 30, 2024

Sehr cooles Projekt @omphteliba ... bei mir zeigt er beim Test zwar die Aufgabe, beim Ausführen kommt aber: Error: No content to summarize for Article ID: 3.

Und bei einem Bild kommt kein Fehler - aber auch keine Description ...

Was mir auffällt, ist, daß im Prompt $prompt_lang drinsteht und das nicht aufgelöst ist?

P.S.: Benutze v0.4 vom 16.8. aus dem "Store"
P.P.S.: Spannend wäre natürlich auch noch, welches Sprachmodell für die beiden Anwendungsfälle am besten geeignet ist

EDITH: Habe gerade auf die Github-Version "Geupdated" - ohne Erfolg. Der "eigentliche" Content liegt in den Slices 3&4 - vielleicht kann das ein Problem sein?

@omphteliba
Copy link
Owner

Hej @Markus-GS,
freut mich, dass es Dir gefällt. Ich bin zzt krank und müsste mir Deine Fragen nächste Woche anschauen. Ich versuche Dir mal so aus der Erinnerung zu antworten:

  • No content to summarize: schau mal in der Datenbank nach, was im Feld "meta_description" drin steht für Deinen Artikel 3. Das klingt so als würde die API keinen Text bekommen.
  • $prompt_lang wird durch die Sprachen ersetzt, die Du auswählst.
  • Slices 3&4: ich nehme eigentlich den ganzen Inhalt der Seite, bereinige den Inhalt dann nur auf den Text
  • Modelle: ja da will ich noch was machen, weil zB nur ein paar Modelle Bilder erkennen können. Aber das bekommt man nicht von OpenAPI über die API, sondern muss man händisch pflegen.

@Markus-GS
Copy link

Markus-GS commented Sep 30, 2024

Hallo @omphteliba Oliver!
Danke für Deine Antwort und Gute Besserung!

So, Kommando zurück :) ... mit YRewrite war ich glaube ich auf dem richtigen Pfad ... Du schreibst ja ins Log :)

30.09.2024, 17:53:43rex_exception Error accessing /ueber/: file_get_contents(/ueber/): Failed to open stream: No such file or directoryDatei: redaxo/src/addons/ff_gpt_tools/lib/GptTools.php:941

Das Problem scheint darin zu liegen, daß die URL kein "File" ist, sondern ein "/ueber/"-"Link" ...

Zusätzlich könnte natürlich auch ein hardcodiertes meta_description problematisch sein, da das Meta INfos AddOn das Feld (inzwischen?) als "art_description" erstellt.

@Markus-GS
Copy link

Bei den Bildern gibt's keine Fehlermeldung - aber auch kein Ergebnis ...

Der Pfad in der Datenbank sieht wie folgt aus:
../media/nextstation5_1536.jpg

Korrekt? Oder sollte da eventuell der ganze drinstehen

@omphteliba
Copy link
Owner

Bei den Bildern gibt's keine Fehlermeldung - aber auch kein Ergebnis ...

Der Pfad in der Datenbank sieht wie folgt aus: ../media/nextstation5_1536.jpg

Korrekt? Oder sollte da eventuell der ganze drinstehen

Der relative Pfad ist korrekt, aber Du musst gpt-4o-mini als Modell nehmen.

@omphteliba
Copy link
Owner

Zusätzlich könnte natürlich auch ein hardcodiertes meta_description problematisch sein, da das Meta INfos AddOn das Feld (inzwischen?) als "art_description" erstellt.

@Markus-GS Du kannst die Felder für die Artikel und Bild Beschreibungen doch frei konfigurieren?!
image

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

No branches or pull requests

3 participants