Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

[camera] Use startVideoCapturing and expose concurrent stream/record #6815

Conversation

adam-harwood
Copy link
Contributor

This uses the new startVideoCapturing implementation, that supports concurrent stream/record.

This is the final piece from #6808.

Closes flutter/flutter#83634

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [relevant style guides] and ran [the auto-formatter]. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
  • I signed the [CLA].
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy], or this PR is [exempt from version changes].
  • I updated CHANGELOG.md to add a description of the change, [following repository CHANGELOG style].
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • All existing and new tests are passing.

This uses the new startVideoCapturing implementation, that supports concurrent stream/record.
@adam-harwood adam-harwood changed the title Use startVideoCapturing and expose concurrent stream/record [camera] Use startVideoCapturing and expose concurrent stream/record Dec 9, 2022
@adam-harwood
Copy link
Contributor Author

The checks will pass once a new version of camera_android, camera_avfoundation, camera_platform_interface, and camera_web are released. What's the timeline of that happening?

@stuartmorgan
Copy link
Contributor

It happens automatically once post-submit tests finish running, so normally it would have happened yesterday, but there was some post-submit test flake that we didn't notice. I'm re-running those now.

@stuartmorgan
Copy link
Contributor

I hit some CI snags with autopublish, but I've now published them manually. Sorry for the delay!

Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Request for secondary review from @stuartmorgan

@jorger5
Copy link

jorger5 commented Jan 2, 2023

Looking forward to have this merged, awesome job @adam-harwood !

@neil-yagao
Copy link

Dude, I am reeeeally looking forward to this feature. Can't wait for this PR to be released. Big thanks for your effort.

Copy link
Contributor

@stuartmorgan stuartmorgan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, it'll just need a merge from main. Sorry for the delay on this last piece, I was on vacation for a few weeks and then have still been catching up.

@adam-harwood
Copy link
Contributor Author

Done, ready for merge I think @stuartmorgan .

@stuartmorgan stuartmorgan added the autosubmit Merge PR when tree becomes green via auto submit App label Jan 18, 2023
@auto-submit auto-submit bot merged commit 11361d0 into flutter:main Jan 18, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 18, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Jan 18, 2023
* 6ef1bc1da Roll Flutter from 8c2fdb8 to cc7845e (2 revisions) (flutter/plugins#6983)

* 56ab33fd1 [shared_pref]: Bump mockito-inline (flutter/plugins#6976)

* e85e0f28f [camera] Allow logical cameras to use all physical cameras via zoom on android 11+ (flutter/plugins#6150)

* 11361d010 [camera] Use startVideoCapturing and expose concurrent stream/record (flutter/plugins#6815)
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…#118682)

* 6ef1bc1da Roll Flutter from 8c2fdb8 to cc7845e (2 revisions) (flutter/plugins#6983)

* 56ab33fd1 [shared_pref]: Bump mockito-inline (flutter/plugins#6976)

* e85e0f28f [camera] Allow logical cameras to use all physical cameras via zoom on android 11+ (flutter/plugins#6150)

* 11361d010 [camera] Use startVideoCapturing and expose concurrent stream/record (flutter/plugins#6815)
mauricioluz pushed a commit to mauricioluz/plugins that referenced this pull request Jan 26, 2023
…lutter#6815)

* Use startVideoCapturing and expose concurrent stream/record

This uses the new startVideoCapturing implementation, that supports concurrent stream/record.

* Ran dart formatter

* retrigger checks

* Account for version bump
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App p: camera
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Camera]: Record video while Image Stream process
5 participants