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

General update to make it work with the new version of OpenALPR (2.3.0) #22

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

sneko
Copy link

@sneko sneko commented Aug 2, 2017

I think it could solve some problems posted in issues :)

@munsterlander
Copy link

@sneko Any chance you can submit this PR to: https://github.com/munsterlander/node-openalpr? I am trying to get this going for a project.

@sneko
Copy link
Author

sneko commented May 30, 2022

Hi @munsterlander , unfortunately it's been 5 years since then 😄

I'm no longer working with OpenALPR projects, and I don't have time to be sure doing a PR somewhere else would work and be meaningful. It's better you take my work and try to publish your own package (or maybe there is another updated repository now in the community?).

EDIT: another solution would to make the main maintainer of this plugin, test & merge my PR, but I guess he is in the same situation than me since then

@munsterlander
Copy link

I saw you you had forked with your PR, so I was going to grab it. OpenALPR is dead as Rekor has gone 100% for profit. I am using it to just get something going to limp by as we need a complete offline setup. Thanks for responding!

@kjlaw89
Copy link
Contributor

kjlaw89 commented May 30, 2022

@munsterlander - Unfortunately it was the same for us. OpenALPR was acquired and the cost to continue using it became too prohibitive to continue using. We ended up switching to platerecognizer.com and hosting a docker image on-site that we send images to for LPR decoding. Pricing ended up better and, at least at the time we switched, it was more accurate. Though, we stopped maintaining the repository once this happened. It'd probably be easier to fork the repository and merge in these extra branches to it to get you by if needed.

@munsterlander
Copy link

Thanks for the info. We also are evaluating PlateRecognizer. That company seems way nicer to work with as well and I have heard / read their studies regarding their accuracy as well. I currently have forked several repos and have something quasi working. Unfortunately, its just not possible to get anything working on Windows only in Electron, but I am doing a hybrid solution with an Android device, websockets, and Electron that is getting it going. Thank you all again for the info!

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