-
Notifications
You must be signed in to change notification settings - Fork 58
[DO NOT MERGE EVER] Gain insights on which component needs to be updated in main from release-1.2.0 #835
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: main
Are you sure you want to change the base?
Conversation
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/' |
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.
[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/' |
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.
[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 \ |
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.
|
||
```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 |
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.
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.
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.
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) |
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.
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 |
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.
Same as above.
DL Streamer Pipeline Server is up to date on main branch and needs no further update |
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: