Skip to content
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

Catalog update #44

Merged
merged 6 commits into from
Jan 28, 2025
Merged

Catalog update #44

merged 6 commits into from
Jan 28, 2025

Conversation

brian-intel
Copy link
Contributor

PR Checklist

What requirement is this design document for?

Merging catalog documentation into the open source documentation

#issue_number

Anything the reviewer should know when reviewing this PR?

If there are any other design Pull Requests or requirements, please link them here (i.e. intel-retail/automated-self-checkout )

Sorry, something went wrong.

Signed-off-by: Brian McGinn <brian.mcginn@intel.com>
Signed-off-by: Brian McGinn <brian.mcginn@intel.com>
Signed-off-by: Brian McGinn <brian.mcginn@intel.com>
Signed-off-by: Brian McGinn <brian.mcginn@intel.com>
@brian-intel brian-intel added documentation Improvements or additions to documentation 3.3 labels Jan 22, 2025
Signed-off-by: Brian McGinn <brian.mcginn@intel.com>
Signed-off-by: Brian McGinn <brian.mcginn@intel.com>
Copy link
Contributor

@ejlee3 ejlee3 left a comment

Choose a reason for hiding this comment

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

LGTM

Do the following to launch the Grafana* dashboard to view the objects being detected and tracked:

1. Open a web browser and enter the following URL to access the Grafana dashboard:
``http://<target_system_IP>:3000``.
Copy link
Contributor

Choose a reason for hiding this comment

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

are we using target_system because the software may be running somewhere else, or why not simplify it to 127.0.0.1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was due to us not being able to use the host network directly and some of the containers needed the host IP to since localhost and 127.0.0.1 didn't work inside the container.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Something we can refactor in a future release once the latest EVAM images are published

@brian-intel brian-intel merged commit bbab261 into intel-retail:main Jan 28, 2025
2 checks passed
@brian-intel brian-intel deleted the catalogUpdate branch January 28, 2025 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.3 documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants