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

PiCamera-based camera provider for Raspberry Pi #5693

Merged
merged 1 commit into from
Apr 11, 2018

Conversation

whitelynx
Copy link
Contributor

Added a new camera provider based on PiCamera, for use with CSI-based cameras on the Raspberry Pi. (not for USB cameras)

@tito
Copy link
Member

tito commented Apr 11, 2018

Nothing to say about the PR, thanks for all!

@tito
Copy link
Member

tito commented Apr 11, 2018

Can you also check the documentation part about the requirements needed for the camera on RPI ? (Numpy + picamera)

@tito tito merged commit 93dd922 into kivy:master Apr 11, 2018
@ghost
Copy link

ghost commented Apr 11, 2018

Seems like numpy is only used to chop off some "bonus" pixels..? Assuming it's not a requirement of PiCamera in the first place, you could maybe avoid it with a "master" texture (at the full/rounded size), and master_texture.get_region(0, 0, cropped_w, cropped_h) for drawing or a Cython snippet?

Related: PR #5210 - seems like it does similar things without external dependencies

@whitelynx
Copy link
Contributor Author

I do like the fact that #5210 doesn't require numpy... I'd be fine with that one replacing mine, if someone more comfortable with the C/Cython side of things could review it.

In the meantime, I'll look at using your suggestion for cutting out the numpy dep in this one :)

@tito
Copy link
Member

tito commented Apr 11, 2018

meh :/ i didn't knew about #5210, why this was wasn't merged :(

@whitelynx whitelynx deleted the feat/camera_picamera branch January 27, 2023 13:30
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

Successfully merging this pull request may close these issues.

None yet

2 participants