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

Fix ths BUG can't save the pictures. #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

QiangJi
Copy link

@QiangJi QiangJi commented Feb 9, 2015

No description provided.

@kylemcdonald
Copy link
Owner

hi! can you please list your:

  1. Mac OSX version
  2. EDSDK version
  3. Canon camera model

and @ofZach does this look similar to something you've run into before?

@ofZach
Copy link

ofZach commented Feb 10, 2015

my issue had to deal with focus failing, and getting stuck in an infinite loop attempting to take a picture. this code helped me:

#24

in the end, I wound up triggering a focus attempt then delaying then taking a photo without AF, that seemed to help alot with speed and stability.

I am still slightly worried about the threading logic, it feels like some function calls need a bit of a chance to report an error or fail gracefully that seems harder to do in the thread.

@QiangJi
Copy link
Author

QiangJi commented Feb 10, 2015

@kylemcdonald

  1. I am using win8.1.
  2. EDSDK version is 2.14;
  3. Canon camera model is EOS1200D

My first time to use.

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.

3 participants