Skip to content

Conversation

ajagadi1
Copy link
Contributor

@ajagadi1 ajagadi1 commented Aug 22, 2025

This PR should never be merged to main. It is only meant to assess if any changes need to go to main from release-1.2.0 branch. Component owners should create separate PRs to update relevant changes from release-1.2.0 into main.

Component owners to update status checklist of which projects are already up to date on main branch:

@ajagadi1 ajagadi1 marked this pull request as draft August 22, 2025 07:53
from jsonschema import validate

DLSTREAMER_URL='https://github.com/open-edge-platform/edge-ai-libraries/tree/main/libraries/dl-streamer/'
DLSTREAMER_URL='https://github.com/open-edge-platform/edge-ai-libraries/tree/release-1.2.0/libraries/dl-streamer/'

Choose a reason for hiding this comment

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

[pylint] reported by reviewdog 🐶
C0301: Line too long (114/100) (line-too-long)


OV_MODEL_ZOO_URL = 'https://github.com/openvinotoolkit/open_model_zoo/tree/master/'
DLSTREAMER_URL='https://github.com/open-edge-platform/edge-ai-libraries/tree/main/libraries/dl-streamer/'
DLSTREAMER_URL='https://github.com/open-edge-platform/edge-ai-libraries/tree/release-1.2.0/libraries/dl-streamer/'

Choose a reason for hiding this comment

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

[pylint] reported by reviewdog 🐶
C0301: Line too long (114/100) (line-too-long)

# Download NLTK corpora (stopwords + punkt)
RUN mkdir -p /opt/share/nltk_data \
# install nltk data
RUN pip install nltk \

Choose a reason for hiding this comment

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

⚠️ [hadolint] <DL3013> reported by reviewdog 🐶
Pin versions in pip. Instead of pip install <package> use pip install <package>==<version> or pip install --requirement <requirements file>


```bash
git clone https://github.com/open-edge-platform/edge-ai-libraries.git
git clone https://github.com/open-edge-platform/edge-ai-libraries.git -b release-1.2.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @ajagadi1 for creating this PR.
I don't think we have to port these changes as these are specific docs changes which should reside only at release-1.2.0 branch.

If you are OK, I can undo these changes in this branch.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @ajagadi1 for clarifying this is not for merge but to assess the changes to see if any changes need to go to main from release-1.2.0 branch.

I've updated the PR checklist, we are good from time series analytics microservice perspective.

- [How to configure custom UDF](https://github.com/open-edge-platform/edge-ai-suites/tree/main/manufacturing-ai-suite/wind-turbine-anomaly-detection/docs/user-guide/how-to-configure-custom-udf.md)
- [Overview.md](https://github.com/open-edge-platform/edge-ai-suites/blob/release-1.2.0/manufacturing-ai-suite/wind-turbine-anomaly-detection/docs/user-guide/Overview.md)
- [Getting Started](https://github.com/open-edge-platform/edge-ai-suites/blob/release-1.2.0/manufacturing-ai-suite/wind-turbine-anomaly-detection/docs/user-guide/get-started.md)
- [How to configure alerts](https://github.com/open-edge-platform/edge-ai-suites/blob/release-1.2.0/manufacturing-ai-suite/wind-turbine-anomaly-detection/docs/user-guide/how-to-configure-alerts.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above

Follow the steps to run the functional tests.
```bash
git clone https://github.com/open-edge-platform/edge-ai-libraries
git clone https://github.com/open-edge-platform/edge-ai-libraries -b release-1.2.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above.

@ajagadi1 ajagadi1 changed the title [DO NOT MERGE] Gain insights on which project needs to be updated in main from release-1.2.0 [DO NOT MERGE EVER] Gain insights on which project needs to be updated in main from release-1.2.0 Aug 22, 2025
@ajagadi1 ajagadi1 requested a review from tjanczak August 22, 2025 09:17
@ajagadi1 ajagadi1 changed the title [DO NOT MERGE EVER] Gain insights on which project needs to be updated in main from release-1.2.0 [DO NOT MERGE EVER] Gain insights on which component needs to be updated in main from release-1.2.0 Aug 22, 2025
@ajagadi1
Copy link
Contributor Author

DL Streamer Pipeline Server is up to date on main branch and needs no further update

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants