Implement Python module for wristband GPIO access

Does what it says on the tin. It will fix #31 (closed)

added 3-Module label

assigned to @rahix

Looks good to me! Thanks for keeping our codebase clean.

However, I'm only a bot - so a human will still have to approve this request.

added 5 commits

• 614f7bb8...af4587a9 - 3 commits from branch card10:master
• 99422553 - feat(epicardium): Add (wristband) GPIO module
• e4b753ee - feat(pycardium): Add (wristband) GPIO module

Compare with previous version

added 2 commits

• bdcbbfc3 - feat(epicardium): Add (wristband) GPIO module
• 32b64570 - feat(pycardium): Add (wristband) GPIO module

Compare with previous version

Thank you very much! Very clean code and the Python facing API looks good, too.

The only nagging point I have is error values. Please return errno codes from all functions in case something goes wrong or invalid values are passed in. On the Python side, raise exceptions when such a value is returned. The easiest way is using

int res = epic_foo();
if (res < 0) {
        mp_raise_OSError(-res);
}

but you might want to use ValueError instead if an invalid argument was passed in (like a pin number which does not exist).

added 2 commits

• 1f00d7be - feat(epicardium): Add (wristband) GPIO module
• 6bf9a6fc - feat(pycardium): Add (wristband) GPIO module

Compare with previous version

I've added documentation for epicardium and pycardium and implemented error handling on the pycardium side as well.

added 1 commit

• 25a54a83 - feat(pycardium): Add (wristband) GPIO module

Compare with previous version

Looks good to me! Thanks for keeping our codebase clean.

However, I'm only a bot - so a human will still have to approve this request.

resolved all threads

Ok, one more thing: Please rebase on latest master, or tick the "Allow commits from members who can merge to the target branch" checkbox so I can do it for you. Then I will merge :)

added 6 commits

• 25a54a83...132874f7 - 4 commits from branch card10:master
• 2fb78caa - feat(epicardium): Add (wristband) GPIO module
• 2dcd065b - feat(pycardium): Add (wristband) GPIO module

Compare with previous version