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

Feature Request: Allow grouping of digits to multiple reading values #123

Closed
papauorg opened this issue Feb 23, 2021 · 3 comments
Closed

Comments

@papauorg
Copy link

This is a really awesome project. I use it for my water meter as described in the documentation. It's working really well and I'm quite happy with your project. Thanks a lot for your work.

Beeing so happy I wanted to go further and use another esp to read my power meter. It would work exactly the same (minus the analog digits) BUT I have a power meter that counts two different values depeding on the time the power is used. It looks similar to this one here:
Powermeter

It would be great if I could use one device for recognizing all the digits and then group them so I can read two values. I could read all digits of both values and merge it to one number, but then I loose the sanity checks that the device performs.

The Idea would be that in the configuration of the ROIs some form of grouping is allowed. E.g. ROI1-6 belong to VALUE1 and ROI7-12 belong to VALUE2 (or some other named group). And those values are then used to be checked and sent via MQTT the same way it's now done for one value. Would this be possible?

Please let me know what you think.

Thanks and best regards,
Philipp

@jomjol
Copy link
Owner

jomjol commented Feb 23, 2021

@papauorg,
I fully get you point. The implementation is fully focused on only one group of digits and analogs only. Therefore this would need a deeper change in the logic. At the moment I'm the only programmer of this and do not have the ressources to do so.

I could offer you maybe another solution: what if you just make the grouping later in a off side postprocessing. So you could specify all digits in one row, make the comma as the separator. So you would have a "virtual" counter number, giving you all digits: abcdef,ghijkl abcdef would be e.g. the upper and ghijkl the lower counters. You would need to take care of the comma in each by yourself.

There are some limits:

  • replacement of the "N" would work
  • but the error correction like MaxRateValue does not make sense

Maybe this helps you.

Best regards,

jomjol

@jomjol
Copy link
Owner

jomjol commented Mar 21, 2021

Due to lack of capacity this is not followed currently. In order to not forget it, it is added to the feature request page: https://github.com/jomjol/AI-on-the-edge-device/blob/master/FeatureRequest.md

Meanwhile this issue will be closed, as it is not active currently.

@jomjol jomjol closed this as completed Mar 21, 2021
@jomjol
Copy link
Owner

jomjol commented Jul 21, 2021

Implemented since v8.0.0

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

No branches or pull requests

2 participants