-
Notifications
You must be signed in to change notification settings - Fork 29
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
Request: Create libuvc separate fork #38
Comments
Hi nsknyc, The libuvc version in this Repo is modified 2 times. First it was modified to work with an older LibUsb versions and working with Android. After that I modified the version again to work with my custom driver input. I think at this point it would be easier to create a new app based on the latest libuvc version to get it running, before getting this merged version sucessfully running with the latest libuvc version. So far, upgrades in the official LibUvc version can also be commited to this libuvc version relativ easily, because the two libuvc versions are still quite related. If you have some suggestions, I`m quite interested in hearing them. Thx, Peter |
Understood! I'll bring over the modifications (from another fork) over to this one. This way we can have radiometric data in this app. Thanks. |
Hello Peter,
I noticed that the libuvc library you have included in the repo has a readme to ktossell github. I checked and that seems to be the old name. Anyways I was wondering if it's possible for you to fork libuvc to keep track of changes. The reason I ask is that groupgets (purethermal) also has some modifications to libuvc, and I'd hate to cause some conflicts. I also don't want to replace your libuvc version with a potentially older version (when I do pull requests). With a fork I can compare the official repo, with your modifications and merge groupgets' as well.
I hope I wasn't too confusing. I have radiometric data working on a test app with Y16 and would like to implement it here. I don't have much experience with other thermal cameras, but I can assume that they function the same way. Maybe this addition will help other.
Thanks!
The text was updated successfully, but these errors were encountered: