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

Device identifier #256

Closed
oocanto opened this issue Aug 26, 2014 · 2 comments
Closed

Device identifier #256

oocanto opened this issue Aug 26, 2014 · 2 comments
Assignees

Comments

@oocanto
Copy link

oocanto commented Aug 26, 2014

Hello, team.

I am using your incredible library and I have to say that you did a wonderful job. Congratulations.

By using your library, I noticed that we do not have an identifier for each device that is discovered in local machine. Regarding to that, I shall make a very dummy question:

Since com.github.sarxos.webcam.ds.buildin.natives.Device class has a getIdentifierStr() method, is it possible to make it accessible from implementations of com.github.sarxos.webcam.WebcamDevice interface across a getId() method by returning a java.lang.String value? I did that locally (Windows XP SP2), and it seems to work fine. It is a simple change in com.github.sarxos.webcam.ds.buildin.WebcamDefaultDevice class.

Being all for now, I look forward to hearing from you soon.

Thank you.

Best regards.

@sarxos
Copy link
Owner

sarxos commented Aug 29, 2014

Hi @oocanto,

I see no issue in adding new method String getId() to WebcamDefaultDevice class, but I have objections in regards to exposing it via WebcamDevice interface. The problem I see is the fact that webcam ID is available if and only if one is using default driver from OpenIMAJ. In case of all other drivers (e.g. GStreamer, FS Webcam, OpenCV and others) only webcam name is available (or address, in OpenCV only). Therefore, adding something non-common to one of the main interfaces is imho questionable.

@sarxos
Copy link
Owner

sarxos commented Oct 19, 2014

I'm closing this ticket as getId() has been added to WebcamDefaultDevice. The WebcamDevice interface remains unchanged.

@sarxos sarxos closed this as completed Oct 19, 2014
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

2 participants