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

feature request: add grab strategies #20

Open
ZacDiggum opened this issue Oct 13, 2016 · 6 comments
Open

feature request: add grab strategies #20

ZacDiggum opened this issue Oct 13, 2016 · 6 comments

Comments

@ZacDiggum
Copy link

Would it be possible to add the supported grab strategies to grab_images()? I noticed a corresponding annotation in factory.pyx.

@mabl
Copy link
Owner

mabl commented Oct 14, 2016

Sure, it's just that I'm driven by my PhD and little short on time... I hope to take a look at it too. I personally like the iterator approach. However, one does not "know" when an iterator is not needed anymore in Python. So I might drop the iterator style API and expose something closer to the metal..

@ZacDiggum
Copy link
Author

Oh, of course. Sorry for my impatience. I'll try to add the strategies myself. By now, the integration of the Basler camera into my existing application works quite well, thanks to PyPylon. There's only a little thread safety thing left but that's about PyQt. When I got this done I could make a small PyQt example GUI as a showcase for PyPylon. Anyways, I wish you success with your PhD. 👍

@ZacDiggum ZacDiggum reopened this Nov 1, 2016
@ZacDiggum
Copy link
Author

Unfortunately I'm not able to add the grab strategies myself. @RichardLangFromNZ, would you be willing to do this?

@RichardLangFromNZ
Copy link

Looks like we need this for the work we're trying to do, so will probably be able to put some time into it and push something back fairly shortly

@RichardLangFromNZ
Copy link

RichardLangFromNZ commented Nov 7, 2016

Fix in PR #23

@ZacDiggum
Copy link
Author

@RichardLangFromNZ
Thanks! Cool! Thanks again! 👍

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