Apparence-io / CamerAwesome

📸 Embedding a camera experience within your own app shouldn't be that hard. A flutter plugin to integrate awesome Android / iOS camera experience.
https://ApparenceKit.dev
MIT License
911 stars 200 forks source link

onImageForAnalysis for multi cam #333

Open lei-cao opened 1 year ago

lei-cao commented 1 year ago

Proposal

Thank you for providing the multi cam feature. Very important for my app. I am testing it, it's really cool. But I realise that onImageForAnalysis is not support at this moment.

Describe your new feature Support onImageForAnalysis with multi cam.

apalala-dev commented 1 year ago

Hi, thank you for your insights!

Multi cam is still in beta and there are a few areas that need to be improved over time. iOS and Android may have different behaviours, please remember to explain on which platform it's not working when posting an issue. 🙏

Image analysis may have a huge impact on performances when displaying multiple cameras, I'm not sure if we'll be able to enable it. On Android, problems often arise when multiplying the use cases (preview + recording video + image analysis is not supported on all devices for instance). We may try to enable it to the main camera first (the one not in picture in picture mode) but that will depend on what is possible on the native side.

istornz commented 1 year ago

I completely agree with @apalala-dev. It's important to acknowledge that while the multi-cam feature is still in beta, there are areas that need improvement.

I understand that image analysis is a great feature to implement but impact performance when multiple cameras are being displayed. Enabling it may not be feasible for now, especially on iOS where problems can arise when combining various use cases such as preview, video recording, and image analysis. It might be worth considering enabling it for the main camera initially.

lei-cao commented 12 months ago

enabling it for the main camera is enough for my case. Hope it could be implemented :) 👍