-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add DSLR Camera Control #5
Comments
Info from @tijaci. Copied from external discussion for archive here.
|
Quick thoughts..
This doesn't sound too bad to me. I agree that a direct gphoto2 module may be best suited for our project needs, but if it's complicated enough for the month remaining in this project... then let's go with subprocess calls. I'm personally cool with subprocess calls if they are low CPU usage and execute quickly. If really necessary we could just use a |
Big Picture wise, I'm imagining that if we can have a class that set's up the setting's like...
And then in the code we use a single API call that we input a name for the to-be-saved photo and it returns the absolute file path of the saved photo. like this..
I think that's all we need for an MVP. |
@Schellenberg3 & I reviewed the camera.py control code and just wanted to ask for a few changes. photogrammetry-capture-motion/camera/camera.py Lines 19 to 25 in 2e0f676
1 - As included in the
2 - I'd also like to document here the file naming structure if we can start that discussion as well. Side Note, |
@acbuynak is this the sort of change you're looking for?
|
I think this looks good. @Schellenberg3 and I will test it tomorrow in the lab. Thanks! |
@tijaci Can you write up some instructions on what base libraries and package(s) we need to install on linux to run this? |
@Schellenberg3 was able to get the commands working via terminal so we know the connection should work now. We just need to purchase a cable long enough for the robot. @tijaci Can you implement the proposed code above into a PR? Or directly into code, either way works. |
@tijaci Demonstrated ability to control a DSLR camera via command line.
Creating this issue to track the creation a control script and addition of this ability to the repo.
The text was updated successfully, but these errors were encountered: