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.

Sign up for GitHub

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

Jump to bottom

Integration tests for AI Viewtools #27959

Closed
12 of 14 tasks
Tracked by #27835
dsilvam opened this issue Mar 13, 2024 · 1 comment · Fixed by #28546, #28545, #28488, #28308 or #28447
Closed
12 of 14 tasks
Tracked by #27835

Integration tests for AI Viewtools #27959

dsilvam opened this issue Mar 13, 2024 · 1 comment · Fixed by #28546, #28545, #28488, #28308 or #28447
Labels
dotCMS : AI Merged QA : Not Needed Release : 24.05.20 Bug- Fixing Team : Falcon Type : Task

Comments

@dsilvam
Copy link
Contributor

dsilvam commented Mar 13, 2024 •
edited by victoralfaro-dotcms

Parent Issue

#27835

Task

  • Write automated tests for all viewtools
    • $ai
      • $ai.generateText
      • $ai.generateImage
    • $ai.search
      • $ai.search.query
      • $ai.search.related
    • $ai.embeddings
      • $ai.embeddings.countTokens
      • $ai.embeddings.generateEmbeddings
      • $ai.embeddings.getIndexCount
      • $ai.embeddings.indexCount
    • $ai.completions
      • $ai.completions.getConfig
      • $ai.completions.config
      • $ai.completions.summarize
      • $ai.completions.raw

Proposed Objective

Core Features

Proposed Priority

Priority 2 - Important

Acceptance Criteria

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response
@dsilvam dsilvam mentioned this issue Mar 13, 2024
Open
@dsilvam dsilvam changed the title Mirate AI Viewtools from plugin to core Migrate AI Viewtools from plugin to core Mar 13, 2024
@dsilvam dsilvam changed the title Migrate AI Viewtools from plugin to core Integration tests for AI Viewtools Mar 14, 2024
@dsilvam dsilvam added Epic and removed Epic labels Apr 10, 2024
@victoralfaro-dotcms victoralfaro-dotcms self-assigned this Apr 17, 2024
victoralfaro-dotcms added a commit that referenced this issue Apr 20, 2024
@victoralfaro-dotcms
#27959: initial commit
8a305e5
victoralfaro-dotcms added a commit that referenced this issue Apr 25, 2024
@victoralfaro-dotcms
#27959: Simplifying even more OpenAI service classes and providing i…
6a166a9 
…ntegration test for AIViewTool class
victoralfaro-dotcms added a commit that referenced this issue Apr 25, 2024
@victoralfaro-dotcms
#27959: Simplifying even more OpenAI service classes and providing i…
6b4ddf0 
…ntegration test for AIViewTool class
victoralfaro-dotcms added a commit that referenced this issue Apr 25, 2024
@victoralfaro-dotcms
#27959: Simplifying even more OpenAI service classes and providing i…
ec0de77 
…ntegration test for AIViewTool class
victoralfaro-dotcms added a commit that referenced this issue Apr 25, 2024
@victoralfaro-dotcms
#27959: Simplifying even more OpenAI service classes and providing i…
69fa96b 
…ntegration test for AIViewTool class
victoralfaro-dotcms added a commit that referenced this issue Apr 25, 2024
@victoralfaro-dotcms
#27959: Simplifying even more OpenAI service classes and providing i…
d73e385 
…ntegration test for AIViewTool class
victoralfaro-dotcms added a commit that referenced this issue Apr 26, 2024
@victoralfaro-dotcms
#27959-search: initial
dfecf04
victoralfaro-dotcms added a commit that referenced this issue Apr 26, 2024
@victoralfaro-dotcms
#27959: Simplifying even more OpenAI service classes and providing i…
e6676d7 
…ntegration test for AIViewTool class
victoralfaro-dotcms added a commit that referenced this issue Apr 26, 2024
@victoralfaro-dotcms
#27959: applying sonarqube feedback
6835349
victoralfaro-dotcms added a commit that referenced this issue Apr 27, 2024
@victoralfaro-dotcms
#27959-search: initial
dbd05ec
victoralfaro-dotcms added a commit that referenced this issue Apr 27, 2024
@victoralfaro-dotcms
#27959-search: initial
2b05637
victoralfaro-dotcms added a commit that referenced this issue Apr 30, 2024
@victoralfaro-dotcms
#27959: Simplifying even more OpenAI service classes and providing i…
5949cba 
…ntegration test for AIViewTool class
victoralfaro-dotcms added a commit that referenced this issue Apr 30, 2024
@victoralfaro-dotcms
#27959: applying sonarqube feedback
c227c44
@dsilvam dsilvam linked a pull request May 1, 2024 that will close this issue
#27959: Adding ITs for AI View Tool main methods. #28308
Merged
github-merge-queue bot pushed a commit that referenced this issue May 1, 2024
@victoralfaro-dotcms
#27959: Adding ITs for AI View Tool main methods. (#28308)
255430f 
* #27959: Simplifying even more OpenAI service classes and providing integration test for AIViewTool class

* #27959: applying sonarqube feedback
@dsilvam dsilvam reopened this May 2, 2024
victoralfaro-dotcms added a commit that referenced this issue May 4, 2024
@victoralfaro-dotcms @vicozizou
#27959-search: adding Embeddings data generation convenience class a…
7f22c76 
…nd SearchToolTest as an integration test
victoralfaro-dotcms added a commit that referenced this issue May 6, 2024
@victoralfaro-dotcms @vicozizou
#27959-search: adding Embeddings data generation convenience class a…
98acde5 
…nd SearchToolTest as an integration test
victoralfaro-dotcms pushed a commit that referenced this issue May 11, 2024
@vicozizou
#27959: creating integration test for CompletionTool.
27d440a
victoralfaro-dotcms pushed a commit that referenced this issue May 13, 2024
@vicozizou
#27959-embeddings: adding integration tests for EmbeddingsTool and r…
15d5dc4 
…efactoring direct calls from Runner and Resource classes to rather use EmbeddingsAPI calls annotated with trasnaction annotations.
victoralfaro-dotcms pushed a commit that referenced this issue May 13, 2024
@vicozizou
#27959-embeddings: fix merge issue
be94b39
victoralfaro-dotcms pushed a commit that referenced this issue May 13, 2024
@vicozizou
#27959-embeddings: adding integration tests for EmbeddingsTool and r…
b1314c4 
…efactoring direct calls from Runner and Resource classes to rather use EmbeddingsAPI calls annotated with trasnaction annotations.
victoralfaro-dotcms pushed a commit that referenced this issue May 13, 2024
@vicozizou
#27959-embeddings: fix merge issue
3b0e958
github-merge-queue bot pushed a commit that referenced this issue May 13, 2024
@victoralfaro-dotcms @vicozizou
#27959-embeddings: adding integration tests for EmbeddingsTool (#28488)
4d6ec45 
And refactoring direct calls from Runner and Resource classes to rather
use EmbeddingsAPI calls annotated with transaction annotations.

---------

Co-authored-by: Victor Alfaro <vico.zizou@gmail.com>
This was linked to pull requests May 13, 2024
#27959-search: adding Embeddings data generation convenience class and SearchToolTest as an integration test #28447
Merged
#27959-embeddings: adding integration tests for EmbeddingsTool #28488
Merged
victoralfaro-dotcms pushed a commit that referenced this issue May 13, 2024
@vicozizou
#27959: creating integration test for CompletionTool.
daf942e
victoralfaro-dotcms pushed a commit that referenced this issue May 14, 2024
@vicozizou
#27959: creating integration test for CompletionTool.
6b52e53
@victoralfaro-dotcms victoralfaro-dotcms mentioned this issue May 14, 2024
Closed
@victoralfaro-dotcms
Copy link
Contributor

PRs:

victoralfaro-dotcms pushed a commit that referenced this issue May 15, 2024
@vicozizou @victoralfaro-dotcms
#27959: creating integration test for CompletionTool.
62d4a50
victoralfaro-dotcms pushed a commit that referenced this issue May 15, 2024
@vicozizou @victoralfaro-dotcms
#27959: creating integration test for CompletionTool.
ed3eca0
victoralfaro-dotcms added a commit that referenced this issue May 15, 2024
@victoralfaro-dotcms
#27959-completions: applying feedback
37e766f
victoralfaro-dotcms added a commit that referenced this issue May 15, 2024
@victoralfaro-dotcms
#27959-completions: applying feedback
e915503
github-merge-queue bot pushed a commit that referenced this issue May 16, 2024
@victoralfaro-dotcms @vicozizou
#27959: creating integration test for CompletionTool. (#28546)
149bea9 
Co-authored-by: Victor Alfaro <vico.zizou@gmail.com>
This was unlinked from pull requests May 16, 2024
#27959: Adding ITs for AI View Tool main methods. #28308
Merged
#27959-search: adding Embeddings data generation convenience class and SearchToolTest as an integration test #28447
Merged
#27959-embeddings: adding integration tests for EmbeddingsTool #28488
Merged
This was linked to pull requests May 16, 2024
#27959: creating integration test for CompletionTool. #28546
Merged
Using init method instead of default constructor #28545
Merged
#27959-embeddings: adding integration tests for EmbeddingsTool #28488
Merged
#27959: Adding ITs for AI View Tool main methods. #28308
Merged
#27959-search: adding Embeddings data generation convenience class and SearchToolTest as an integration test #28447
Merged
@dsilvam dsilvam closed this as completed May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dotCMS : AI Merged QA : Not Needed Release : 24.05.20 Bug- Fixing Team : Falcon Type : Task
Projects
dotCMS - Product Planning
Status: Done
Milestone
No milestone
2 participants
@dsilvam @victoralfaro-dotcms