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

Support multiple cameras #4

Open
electricshaman opened this issue Jun 3, 2017 · 9 comments
Open

Support multiple cameras #4

electricshaman opened this issue Jun 3, 2017 · 9 comments

Comments

@electricshaman
Copy link
Contributor

No description provided.

@fhunleth
Copy link
Collaborator

fhunleth commented Feb 2, 2018

@electricshaman or @hsztul If I take a look at adding multiple camera support to the C side, can one of you make the updates to the Elixir side?

@hsztul In your device, are you using two cameras simultaneously or is the second camera port more convenient for some reason? Also, what's the cheapest or easiest way for Jeff and me to try out the second camera port? It's only accessible via the compute module, right?

@electricshaman
Copy link
Contributor Author

@fhunleth Sure, if I were to do it, the ideal situation would be to make camselect a parameter to start_link then return a PID that the client would have to pass into everything. This would break backwards compatibility, of course.

@hsztul
Copy link

hsztul commented Feb 3, 2018

@fhunleth I am using both camera ports simultaneously using the compute module.
I believe that the only way to test this would be to use the compute module (+dev kit).
I have one and could help test.

I can try helping on the Elixir side (relatively new to Elixir, so might have to bear with me).

@fhunleth
Copy link
Collaborator

fhunleth commented Feb 3, 2018

@electricshaman That's the only route that I can think of doing to implement this.

@hsztul Welcome to Elixir, then! We'll definitely be leaning on you to test. Also, what @electricshaman is proposing for how to change the Elixir API isn't too bad. It's mostly a mechanical change. I think that I'll have a chance to take a look at the C code either tomorrow or Monday. I'll post when I have something to try.

@fhunleth
Copy link
Collaborator

fhunleth commented Feb 4, 2018

@hsztul Totally untested code is available in the dual-camera branch. I hardcoded it to use the second camera. Just look for the FIXME. Fingers are crossed that this works...

@fhunleth
Copy link
Collaborator

@hsztul @electricshaman Just curious if anyone was going to follow up on the dual camera change?

@electricshaman
Copy link
Contributor Author

@fhunleth I don't have a compute module or I would try it.

@electricshaman
Copy link
Contributor Author

@hsztul Have you had a chance to test this?

@fhunleth
Copy link
Collaborator

FYI - I met up with the Bowery Farming team over the weekend and found out that they abandoned using picam and just call raspi-still directly (they don't need video). I personally think that this work is so close that someone with a compute module and two cameras could easily finish it off.

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

No branches or pull requests

3 participants