intel-retail / loss-prevention

Apache License 2.0
0 stars 4 forks source link

feat: add/update benchmark of loss prevention pipelines documentation and make targets #16

Closed jim-wang-intel closed 3 weeks ago

jim-wang-intel commented 3 weeks ago

PR Checklist

What are you changing?

Issue this PR will close

close: #15

Anything the reviewer should know when reviewing this PR?

Test Instructions if applicable

If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/loss-prevention )

jim-wang-intel commented 3 weeks ago

On my run, I had to specify python3 in the make targets and in the benchmark-scripts target init-packages I had to force the packages to install with --break-system-packages since I am not using a venv...

Thanks @seanohair22 for letting us know: the existing documentation suggests we use vent in Prerequisite section; but that is a good testing for non-virtual env. Usually I would also suggest running in venv so that if somehow python is messed up, you are less miserable :-) .