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

ER: Allow server to accept IOSurfaceRef directly #11

Open
pizthewiz opened this issue Jan 13, 2014 · 5 comments
Open

ER: Allow server to accept IOSurfaceRef directly #11

pizthewiz opened this issue Jan 13, 2014 · 5 comments

Comments

@pizthewiz
Copy link

Not for naught, Syphon uses an IOSurfaceRef internally and it would be great if I could provide it directly instead of converting it to a texture first.

@vade
Copy link
Member

vade commented Jan 13, 2014

TO be clear, you would want public API access to the internal IOSurface rather than the vended SyphonImage* pointer as a textureID associated with a context?

Can you be clear about your use case, and what / where you are limited now (just so we know).

@pizthewiz
Copy link
Author

Sorry the other way around, I want to publish an IOSurfaceRef to a server, not pull one out of a client (though others might find that useful). In my specific use, I am pulling an IOSurfaceRef from a display stream and wonder if it would be advantageous to publish it to a SyphonServer without a texture conversion.

@bangnoise
Copy link
Member

Not wholly against this, but two considerations if this becomes a thing:

  1. Syphon conceptually presents a single live surface rather than a series of frames. Internally we reuse a single IOSurface (until frame resize) which gives it a considerable speed boost and cuts down on IPC
  2. Syphon in the future may not always use IOSurface internally (eg over a network) and we should be mindful of adding complexity in that case

@pizthewiz
Copy link
Author

My use case might be super specific and in turn, not worth the risk of exposing Syphon's internal details if that compromises or conflicts with future plans.

@bangnoise
Copy link
Member

If so inspired you could create a modified version of the framework for your application - such a framework would work fine with existing clients.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants