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

🐛 Can not save video > 1minute on android #1365

Closed
3 of 4 tasks
nhuthuynh195 opened this issue Dec 1, 2022 · 4 comments · Fixed by #1466
Closed
3 of 4 tasks

🐛 Can not save video > 1minute on android #1365

nhuthuynh195 opened this issue Dec 1, 2022 · 4 comments · Fixed by #1466
Labels
🐛 bug Something isn't working

Comments

@nhuthuynh195
Copy link

nhuthuynh195 commented Dec 1, 2022

What were you trying to do?

When record video > 1 minute on android not save to device (Android 13)

Reproduceable Code

No response

What happened instead?

When record video > 1 minute on android not save to device, when I use RNFS try read path will return error file not found.

Relevant log output

No response

Device

OnePlus 8T

VisionCamera Version

2.15.2

Additional information

image

@nhuthuynh195 nhuthuynh195 added the 🐛 bug Something isn't working label Dec 1, 2022
@nhuthuynh195
Copy link
Author

Anyone can resolve issue?

@mrousavy
Copy link
Owner

mrousavy commented Dec 6, 2022

Hey! Oof, this looks like a native CameraX issue. This is Google's fault. Can you create an issue in their issue tracker?

@mrousavy
Copy link
Owner

mrousavy commented Dec 6, 2022

Actually wait - there is no error message here. This just says that the Camera has been destroyed (e.g. when you move to a new screen).

Can you show the full log output (scroll further up), maybe there's something interesting there?

@myselfuser1
Copy link

@mrousavy
Copy link
Owner

Hey! I've rewritten the entire Android codebase of VisionCamera from CameraX to Camera2 in the efforts of ✨ VisionCamera V3.

I just now completed the Camera2 rewrite and I believe the core structure is running, but there might be some edge cases to iron out. Can you try and test the PR #1674 for me to see if you can still reproduce this issue here?

Here's an instruction on how you can test that: #1674 (comment)

If the issue cannot be reproduced with that version/PR anymore, then hoorayy, I fixed it! 🎉
Otherwise please let me know and I'll keep this issue open to keep track of it.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants