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

🚚 (#266): RESTify az-document-ai service #279

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

byte-sized-emi
Copy link
Contributor

closes #266

@byte-sized-emi byte-sized-emi self-assigned this Jan 26, 2025
@byte-sized-emi byte-sized-emi marked this pull request as draft January 26, 2025 14:09
Copy link
Contributor

@3thr3n 3thr3n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wollten wir nicht direkt das Dokument an den AZ-Dokumenten-AI-Service schicken? Wir machen jetzt immer noch mit URLs

import org.kie.kogito.internal.process.runtime.KogitoProcessInstance;

@ApplicationScoped
public class AnalyzeDocumentDelegate {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sollte das nicht über den FIN-Service gehen?
Der Service uploaded doch das Dokument. Der Org-Service kann doch damit nichts anfangen.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wie meinst du über den FIN-Service gehen? Idee momentan ist das dieses Delegate vom FIN-Service das Dokument runterlädt (weil es im kogito process nicht vorhanden ist, nur die URL), um es dann an den az-document-ai service weiterzugeben

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das Hochladen und dann verarbeiten wird doch über den FIN-Service ausgelöst warum sollten wir dann im Org-Service das als ProzessDelegate haben?
So macht das auch Markus in #217.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das musst du @d135-1r43 fragen, wird sein weil der Prozess aus #217 im org-service implementiert ist (soweit ich weiß)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Genau, das mache ich transparent im Fin-Service, actually sogar direkt beim Uploaden des Belegbildes. Kann also raus aus dem Org Service.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also trotzdem als delegate, aber dafür im Fin Service? Haben wir im Fin Service im Prozess die Datei rumliegen?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • FIN has endpoint for document upload
  • this endpoint starts a kogito process, which extracts data from document
  • process has file or url in it

@byte-sized-emi
Copy link
Contributor Author

Ja; deswegen ist der PR noch im draft. Ich hab noch auf confirmation gewartet wie genau wir des machen, und die Implementierung davon muss auch noch auf meine Prüfungsphase warten.

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

Successfully merging this pull request may close these issues.

Restify Azure AI
3 participants