wip: import analysis result#119
Closed
duaakb wants to merge 8 commits intotexttechnologylab:developfrom
Closed
Conversation
Myyyvothrr
reviewed
Sep 18, 2025
| AnalysisSession s = getCachedSession(analysisId); | ||
| if (s == null) throw new IllegalArgumentException("No cached session for id: " + analysisId); | ||
|
|
||
| java.io.ByteArrayOutputStream bos = new java.io.ByteArrayOutputStream(); // Convert JCas -> XMI bytes |
Member
There was a problem hiding this comment.
This seems to be duplicated, please reuse the function from above
Myyyvothrr
reviewed
Sep 18, 2025
| if (documentId != null && !documentId.isBlank()) fields.put("documentId", documentId); | ||
| if (casView != null && !casView.isBlank()) fields.put("casView", casView); | ||
|
|
||
| String corpusConfigJson = System.getenv("UCE_CORPUS_CONFIG_JSON"); // Include corpusConfig |
Member
There was a problem hiding this comment.
I think we should not need the corpus config here, please check
Myyyvothrr
reviewed
Sep 18, 2025
| RunDUUIPipeline.AnalysisResponse resp = | ||
| pipeline.getModelResourcesWithHandle(selectedModels, inputText, inputClaim, | ||
| inputCoherence, inputStance, inputLLM); | ||
| DUUIInformation DataRequest = pipeline.getModelResources(selectedModels, inputText, inputClaim, inputCoherence, inputStance, inputLLM); |
Member
There was a problem hiding this comment.
Cant you use the result from "resp" here?
Myyyvothrr
reviewed
Sep 18, 2025
| // send to importer | ||
| long corpusId = Long.parseLong(ctx.queryParam("corpusId")); // from ?corpusId=... | ||
| RunDUUIPipeline.sendToImporterViaHttp( | ||
| "http://localhost:4567/api/ie/upload/uima", |
Member
There was a problem hiding this comment.
Please check if we can use only "/api/ie/..." here as above, we should not hard-code this
Member
|
Continued in #132 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Analysis results are cached for 45 minutes.
When the user clicks “Save CAS”, the analysis result as an XMI file is sent to the importer.
More details will follow