micropython-mfrc522 #4

Merged
stefank merged 11 commits from micropython-mfrc522 into main 2025-01-28 18:33:06 +00:00
Member

Await-able python-only implementation with selectable poll rate.

Await-able python-only implementation with selectable poll rate.
stefank added 6 commits 2024-11-26 21:04:35 +00:00
Added micropython library for the mfrc522 card reader.
All checks were successful
Check code formatting / Check-C-Format (push) Successful in 32m45s
Check code formatting / Check-Python-Flake8 (push) Successful in 9s
Run unit tests on host / Run-Unit-Tests (push) Successful in 17s
716731180e
Added deployment script and example main.py.
Some checks failed
Check code formatting / Check-C-Format (push) Successful in 7s
Check code formatting / Check-Python-Flake8 (push) Failing after 9s
Run unit tests on host / Run-Unit-Tests (push) Successful in 16s
4fbd64a51b
Made tag uid retrieval awaitable.
Some checks failed
Check code formatting / Check-C-Format (push) Successful in 51s
Check code formatting / Check-Python-Flake8 (push) Failing after 10s
Run unit tests on host / Run-Unit-Tests (push) Successful in 8s
3ea4801d1b
Fixes for flake8.
All checks were successful
Check code formatting / Check-C-Format (push) Successful in 7s
Check code formatting / Check-Python-Flake8 (push) Successful in 8s
Run unit tests on host / Run-Unit-Tests (push) Successful in 17s
0b8839596a
matthias requested changes 2024-12-03 18:24:41 +00:00
Dismissed
@@ -0,0 +1,33 @@
#!/usr/bin/bash
Owner

Please don't do it like this, include the module in the manifest.py in software/boards/RPI_PICO_W/ to include it in our micropython environment.

Please don't do it like this, include the module in the manifest.py in software/boards/RPI_PICO_W/ to include it in our micropython environment.
matthias marked this conversation as resolved
@@ -0,0 +4,4 @@
def uid_to_string(uid: list):
uid_string = "0x"
for i in uid:
Owner

Maybe something like

return "0x" + "".join([f'{i:02x}' for i in uid])

?

Maybe something like ```python return "0x" + "".join([f'{i:02x}' for i in uid]) ``` ?
matthias marked this conversation as resolved
@@ -0,0 +11,4 @@
async def get_tag_uid(reader: MFRC522, poll_interval_ms: int = 50) -> list:
while True:
reader.init()
Owner

TODO: We should check that the maximal latency introduced by these "blocking" functions (reader.*) in async context is not too high.

TODO: We should check that the maximal latency introduced by these "blocking" functions (reader.*) in async context is not too high.
stefank marked this conversation as resolved
stefank added 2 commits 2024-12-03 20:25:09 +00:00
Being more pythonesk.
All checks were successful
Check code formatting / Check-C-Format (push) Successful in 31m2s
Check code formatting / Check-Python-Flake8 (push) Successful in 9s
Run unit tests on host / Run-Unit-Tests (push) Successful in 8s
3c81b916ea
stefank added 1 commit 2025-01-14 19:45:55 +00:00
Usable mfrc522 code.
Some checks are pending
Check code formatting / Check-C-Format (push) Waiting to run
Check code formatting / Check-Python-Flake8 (push) Waiting to run
Run unit tests on host / Run-Unit-Tests (push) Waiting to run
a613a18097
Not as async as possible, and definately needs a future revisit. But
works for now, with acceptable delays, if the newly introduced
tocard_retries parameter of class MFRC522 is used and set to a
reasonable low value.
stefank added 1 commit 2025-01-14 19:46:52 +00:00
Using relative paths for submodules.
Some checks failed
Check code formatting / Check-C-Format (push) Successful in 7s
Check code formatting / Check-Python-Flake8 (push) Failing after 10s
Run unit tests on host / Run-Unit-Tests (push) Successful in 1m57s
18742d0c13
stefank force-pushed micropython-mfrc522 from 18742d0c13 to 7213230cad 2025-01-14 19:50:07 +00:00 Compare
stefank added 1 commit 2025-01-28 18:24:35 +00:00
Documented maximum observed delay for future reference.
All checks were successful
Check code formatting / Check-C-Format (push) Successful in 6s
Check code formatting / Check-Python-Flake8 (push) Successful in 9s
Run unit tests on host / Run-Unit-Tests (push) Successful in 8s
c68942fe24
stefank force-pushed micropython-mfrc522 from c68942fe24 to f213a51e36 2025-01-28 18:27:34 +00:00 Compare
matthias approved these changes 2025-01-28 18:32:17 +00:00
stefank merged commit 73d107f927 into main 2025-01-28 18:33:06 +00:00
stefank deleted branch micropython-mfrc522 2025-01-28 18:33:07 +00:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: TonBERRY/tonberry-pico#4