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

Meter detection not yet implemented? #5

Open
2ni opened this issue Jun 11, 2023 · 1 comment
Open

Meter detection not yet implemented? #5

2ni opened this issue Jun 11, 2023 · 1 comment
Assignees
Labels
Seeed_Arduino_GroveAI Label for Seeed_Arduino_GroveAI UAY Unassigned yet

Comments

@2ni
Copy link

2ni commented Jun 11, 2023

Describe the bug
I tried to configure the GroveAi in circuitpython. The face detection example seems to work, but the meter not.
Whenever I try to set the algorithm to meter reading ([0xA0, 0x01, 0x03]), it doesn't save it. When reading the algo out, it keeps the one which was set before.
Also when I set the model to meter reading ([0xA0, 0x11, 0x13]), I get a timeout error from the device.

To Reproduce
Steps to reproduce the behavior:

  1. set algo to 0x03 (meter reading) -> not saved, keeps algo which was set beforehand
  2. set model to 0x13 (meter reading) -> timeout, can only set it to 0x00 or 0x11. Any other value results in a timeout error
    Code:
result = read(bytes([0x80, 0x01]), 2)
printhex("firmware", result)
# set algo (meter: 0x03)
write(bytes([0xA0, 0x01, 0x03])) # can only set 0=object detection,1=object counting,2=classification
result = read(bytes([0xA0, 0x00]), 1)
printhex("algo (expected 0x03):", result)

# set model (meter: 0x13 according to Protocol.md, 0x01 according to meter_reading.ino. Both fail)
write(bytes([0xA0, 0x11, 0x01]))
result = read(bytes([0xA0, 0x10]), 1)
printhex("model (0x01):", result)

Output:

firmware 0x01 0x30
algo (expected 0x03): 0x01
Traceback (most recent call last):
  File "code.py", line 58, in <module>
  File "code.py", line 31, in read
OSError: [Errno 116] ETIMEDOUT

The Protocol.md states model=0x13 for meter reading, but according to Seeed_Arduino_GroveAI.h there's no 0x13. And the example meter_reading.ino sets model=0x01 for meter reading. But this also leads to a timeout.

I fear that the meter reading "classification" is not yet deployed to the latest code version?

Expected behavior
I would expect to be able to set the algo and model according to the doc or example for meter reading without leading to an error.

@MatthewJeffson MatthewJeffson added UAY Unassigned yet Seeed_Arduino_GroveAI Label for Seeed_Arduino_GroveAI labels Oct 9, 2024
@Lesords Lesords self-assigned this Oct 14, 2024
@Lesords Lesords removed their assignment Oct 16, 2024
@LynnL4
Copy link
Member

LynnL4 commented Oct 18, 2024

Hi, have you tried upgrading the firmware, are the other modes normal. https://github.com/Seeed-Studio/Seeed_Arduino_GroveAI/releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Seeed_Arduino_GroveAI Label for Seeed_Arduino_GroveAI UAY Unassigned yet
Projects
Status: Todo
Development

No branches or pull requests

4 participants