Skip to content

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

Open
wants to merge 10 commits into
base: solutions-poc
Choose a base branch
from

Conversation

Rakshit1798
Copy link

@Rakshit1798 Rakshit1798 commented May 8, 2025

Description

Migrate the Gen-AI Summarization Video
Industry Solutions article to docs.

Staging

Rakshit1798 and others added 8 commits May 1, 2025 20:36
* 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
Copy link

netlify bot commented May 8, 2025

Deploy Preview for docs-atlas-architecture ready!

Name Link
🔨 Latest commit 147687d
🔍 Latest deploy log https://app.netlify.com/sites/docs-atlas-architecture/deploys/681c79650777f800086050b1
😎 Deploy Preview https://deploy-preview-217--docs-atlas-architecture.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@jeff-allen-mongo jeff-allen-mongo left a 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:
Copy link
Collaborator

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

Suggested change
.. _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
Copy link
Collaborator

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.

Suggested change
Gen AI-powered video summarization
Gen AI-Powered Video Summarization

Comment on lines +76 to +78
-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
Copy link
Collaborator

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.

Suggested change
-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.

Comment on lines +127 to +130
.. code-block:: sh

{
“videURL”: “https://youtu.be/exampleID”,
Copy link
Collaborator

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:

Suggested change
.. 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…”,
Copy link
Collaborator

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?

Comment on lines +133 to +135
.. card-group::
:columns: 2
:style: extra-compact
Copy link
Collaborator

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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra colon

Suggested change
**Retail:**: Telecommunications is at the heart of edge and
**Retail:** Telecommunications is at the heart of edge and

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.

3 participants