-
Notifications
You must be signed in to change notification settings - Fork 30
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
Added support for Apple devices #65
base: main
Are you sure you want to change the base?
Conversation
So, the current pull request lacks of one important thing for OSX - change of |
MEMO: If sync-over-async is not going to be a fatal problem, we could add |
@YohDeadfall Perhaps a brief description of how to test it on a Mac would be better for people to feel free to try it 😄 |
Added it, but @kiminuo can provide a little bit more on it since he helped me originally. |
I simply ran
then an Avalonia application is supposed to appear. One selects a camera and that's basically it. If you have multiple .NET versions on your machine, it's good to add {
"sdk": {
"version": "6.0.100",
"allowPrerelease": false,
"rollForward": "latestFeature"
}
} but I'm not sure what .NET versions should be supported so it's just a general remark. |
I wasn't even sure if the Avalonia version would actually work on the Mac, so thanks for the help! |
It works perfectly on all platforms and even might work on Tizen. At least there's an issue on it and some work is being done. |
@YohDeadfall Not much of a response, is it running with no problems at all? :) |
The problem here is that it crashes trying to process a frame. @kiminuo did testing, but you know, when development happens by one developer and testing by another it takes too much time and energy. There's a Xamarin project for barcode scanning I can use to make a debug version with a single hard coded characteristic and the most simple image format. Then if it runs without any issue, we can publish it and bring features later when there's some interest. Is it okay? |
Oh... I would have been able to debug it myself if I could get a Mac... |
@YohDeadfall Hi, I am now trying to extend the transcode type for this issue. Specifically, a modification that would change the So, after much thought, I'd still like to accept this PR once and for all before this changing, what do you think? I'm also wondering if it would be better to put a notation in the documentation that says "This is experimental and behavior verification and modifications are welcome" so that people who want to implement captures can see it and try it out. |
Here's a start you might want to consider be698df |
The current problem is that when this pull request is tested it causes a crash due to finalization of an If someone is able to continue on that, I would be happy to provide any other information and help in dealing with it. But for now I have no Mac device and cannot test it myself. |
Here's my take on supporting Apple devices including desktop and mobile. It's not completed yet, but should be close to that point.
Some dependencies I changed to make it easier for me to develop, so they can be reverted afterwards when finishing the feature.
Fixes #45.
Testing
To test changes
FlashCap.Avalonia
insamples
can be used. That application has just two combo boxes to select a web cam and then characteristics.