-
Notifications
You must be signed in to change notification settings - Fork 30
AI video summarization #217
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
base: solutions-poc
Are you sure you want to change the base?
AI video summarization #217
Conversation
* add event-driven-inventory solutions library * added file to appropriate structure * added files for correct structure * added solutions overview section * added video * added images into Architecture * added card to solutions library page * added urls * added reference architecture section * added data model section * added key learnings, technologies and authors * redeployed asset * added building the solution section * changed index name for the SL * changed order sl alphabetically * updated index name * updated index name * updated index name * changed files names * updated url * added path to retail-catalog * updated path * updated path * changed url change streams * updated file name for this solution * added image sizes to figures * addressed git review suggestions * changed headings solutions overview * removed punctuation from figures * resolved conflicts
✅ Deploy Preview for docs-atlas-architecture ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Rakshit1798 thanks for this update. I left a few comments.
@@ -0,0 +1,290 @@ | |||
.. _arch-center-is-Gen AI-powered-video-summarization-solution: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove space from the reference
.. _arch-center-is-Gen AI-powered-video-summarization-solution: | |
.. _arch-center-is-Gen-AI-powered-video-summarization-solution: |
.. _arch-center-is-Gen AI-powered-video-summarization-solution: | ||
|
||
================================== | ||
Gen AI-powered video summarization |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest title case for titles and section headings.
Gen AI-powered video summarization | |
Gen AI-Powered Video Summarization |
-To this end, the application integrates Optical Character | ||
Recognition (OCR) and Artificial Intelligence (AI) to perform | ||
real-time code analysis directly from the video frames. This |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indentation on this block needs to be changed. Is it meant to be a sub-bullet of the one above? If so, the lines after the first one need to be indented so they line up with the text in the first line.
-To this end, the application integrates Optical Character | |
Recognition (OCR) and Artificial Intelligence (AI) to perform | |
real-time code analysis directly from the video frames. This | |
- To this end, the application integrates Optical Character | |
Recognition (OCR) and Artificial Intelligence (AI) to perform | |
real-time code analysis directly from the video frames. This |
And similar indentation for the remaining lines in the block.
.. code-block:: sh | ||
|
||
{ | ||
“videURL”: “https://youtu.be/exampleID”, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indentation here needs to be changed. .. code-block:: sh
needs to start on column 1, and the contained code needs to be shifted accordingly:
.. code-block:: sh | |
{ | |
“videURL”: “https://youtu.be/exampleID”, | |
.. code-block:: sh | |
{ | |
“videoURL”: “https://youtu.be/exampleID”, |
There is also a typo here - videURL
should be videoURL
.
“Length”: “1533s”, | ||
“Thumbnail”: “https://exmpl.com/thumb.jpg” | ||
}, | ||
“transcript”: “Full transcript…”, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The quotes in this code-block are "smart quotes". Can we replace with regular quotation marks?
.. card-group:: | ||
:columns: 2 | ||
:style: extra-compact |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We already have a card-group
for Gen AI on this tab, so you can just put your card inside the pre-existing group. If you delete these three lines, it should work and look as expected.
productivity and learn new technologies faster with a GenAI video | ||
summarization solution. | ||
|
||
**Retail:**: Telecommunications is at the heart of edge and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extra colon
**Retail:**: Telecommunications is at the heart of edge and | |
**Retail:** Telecommunications is at the heart of edge and |
Description
Migrate the Gen-AI Summarization Video
Industry Solutions article to docs.
Staging