[Marketplace Contribution] Google Vertex AI #27010
Merged
Conversation
This file contains 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
content-bot
changed the base branch from
master
to
contrib/xsoar-contrib_sepaprivate-contrib-GoogleVertexAI
I don't know how to add images to the readme so will look into it later
|
Thank you for your contribution. Your generosity and caring are unrivaled! Rest assured - our content wizard @thefrieddan1 will very shortly look over your proposed changes. |
thefrieddan1
added
Community
Community Support Level
There was a problem hiding this comment.
Hi @sepaprivate thank you for your contribution we appreciate your effort, I reviewed the content in the PR and would like to provide some feedback:
Packs/GoogleVertexAI/Integrations/GoogleVertexAI/GoogleVertexAI.py
Outdated
Show resolved
Hide resolved
Packs/GoogleVertexAI/Integrations/GoogleVertexAI/GoogleVertexAI.py
Outdated
Show resolved
Hide resolved
thefrieddan1
added
the
pending-contributor
Changes as per pull request reviewer's feedback
|
All required changes have been applied. @thefrieddan1 thank you for your suggestions |
…I' into sepaprivate-contrib-GoogleVertexAI
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
remove print statement
Adding Images to readme
Fix Images paths
…I' into sepaprivate-contrib-GoogleVertexAI
|
Readme is updated now with images |
replace images with Doc_files
Fix yml structure
move doc_files to integration folder
Fix paths with new doc_files folder moved to integration folder
Fix Flake8 issues and replaced Demisto.log with Demisto.debug
Fix unused variable
…I' into sepaprivate-contrib-GoogleVertexAI
Fixed Flake8 Errors
Change images paths and names
…ectory No need for that folder anymore
thefrieddan1
added
ready-for-instance-test
Fix test_module
thefrieddan1
approved these changes
May 30, 2023
Fixing Flake8 issues
thefrieddan1
merged commit 0b7f77a
into
demisto:contrib/xsoar-contrib_sepaprivate-contrib-GoogleVertexAI
4 tasks
thefrieddan1
pushed a commit
that referenced
this pull request
May 30, 2023
* "pack contribution initial commit" * Update README.md I don't know how to add images to the readme so will look into it later * Update README.md * Update README.md * Delete GoogleVertexAI_image.png * Fix Logo * Update pack_metadata.json * Update GoogleVertexAI.yml * Update GoogleVertexAI.py * Update GoogleVertexAI.yml * Update GoogleVertexAI.yml * Update GoogleVertexAI.py * Update GoogleVertexAI.py * Update GoogleVertexAI.py Changes as per pull request reviewer's feedback * Update GoogleVertexAI.py remove print statement * Update README.md Adding Images to readme * Update README.md Fix Images paths * Add Images folder for readme * Delete Packs/GoogleVertexAI/images directory * Add files via upload * Update README.md replace images with Doc_files * Update GoogleVertexAI.yml Fix yml structure * Delete Packs/GoogleVertexAI/Doc_files directory move doc_files to integration folder * Move Doc_files to integration folder * Update README.md Fix paths with new doc_files folder moved to integration folder * Update GoogleVertexAI.py Fix Flake8 issues and replaced Demisto.log with Demisto.debug * Update GoogleVertexAI.py Fix unused variable * Update GoogleVertexAI.py Fixed Flake8 Errors * Update GoogleVertexAI.py * Update README.md Change images paths and names * Delete Packs/GoogleVertexAI/Integrations/GoogleVertexAI/Doc_files directory No need for that folder anymore * Update GoogleVertexAI.py Fix test_module * Update GoogleVertexAI.py * Update GoogleVertexAI.py Fixing Flake8 issues --------- Co-authored-by: xsoar-bot <[email protected]> Co-authored-by: sepaprivate <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Status
Contributor
@sepaprivate
Description
Fine-tuned to conduct natural conversation. Using Google Vertex Ai (PaLM API for Chat) The current integration of Google Vertex Ai is focusing only on the Generative AI model (PaLM) using the Chat prediction. Later, this plugin will be updated to include the following:
Notes
I will update the guide so you can test it as it will require a lot of steps to be configured
Auto-Generated Documentation Requiring Modification
Video Link
Short demo video of the Pack usage. Speeds up the review. Optional but recommended. Use a video sharing service such as Google Drive or YouTube.