max86150 pycardium api
MAX86150 Epicardium was merged with !371 (merged) as continuation of !194 (closed). Goal of the merge request is to provide stable MAX86150 pycardium API.
Merge request reports
Activity
added 3-Module 6 - Needs Review labels
assigned to @rahix
mentioned in merge request !194 (closed)
has anyone tested this code recently? originally when I wrote it, the stream died the moment the fifo filled up (so enabling the sensor but not immediately starting to read from the sensor broke it until a reboot) and I'm not sure if this was a bug of the card10's stream implementation or my code but I haven't followed firmware development for a while now...
Ugh, that might very well be related to our streams ... They badly need a rework. We had a similar issue with the other streams for which we created a workaround in commit d974cd2f ("hack(streams): Discard overflowing samples"). That might be applicable here, too, until we rewrite the entire streams API.
- Resolved by schneider
Yes, latest version I tried was quite stable:
a. spo2.elf delivers expected waves on every 2-3 start (otherwise spo2.elf just draws initial line and stops).
Further work as I see it:
- analyze and fix "a."
- As @rahix suggests, do similar to d974cd2f changes. It should improve current python bindings behavior.
- compute spo2 (I propose to do it in other merge request).
Currently, I'm blocked with #189 (closed): could not flash and need debugger to continue...
Edited by Arist - Last reply by schneider
added 6 commits
-
085a6a42...fbeb7766 - 5 commits from branch
card10:master - 0663aaee - Merge remote-tracking branch 'card10/master'
-
085a6a42...fbeb7766 - 5 commits from branch
