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

React native vb #1

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

React native vb #1

wants to merge 10 commits into from

Conversation

yavuzcakir
Copy link
Collaborator

No description provided.

@@ -15,7 +15,7 @@ Everyone is welcome to our [Discourse community](https://react-native-webrtc.dis

## WebRTC Revision

* Currently used revision: [M100](https://github.com/jitsi/webrtc/releases/tag/v100.0.0)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

please check latest jitsi webrtc version

@@ -141,13 +141,6 @@ - (instancetype)initWithDelegate:(__weak id<RTCVideoCapturerDelegate>)delegate {
return self;
}

- (void)setConnection:(SocketConnection *)connection {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

why did you remove it

@@ -96,6 +100,17 @@ - (void)startCapture {
dispatch_semaphore_t semaphore = dispatch_semaphore_create(0);

__weak VideoCaptureController *weakSelf = self;

if (self.vb) {
// Selfie Segmentator library requires kCVPixelFormatType_32BGRA format
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

please update comment with MLKit

NSDictionary *videoContraints = constraints[@"video"];
RTCCameraVideoCapturer *videoCapturer;

NSLog(@"Video constraint in create video track: %@", videoContraints);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Please change NSLog with RCTLog

@@ -114,6 +130,9 @@ - (RTCVideoTrack *)createScreenCaptureVideoTrack {
RCT_EXPORT_METHOD(getUserMedia:(NSDictionary *)constraints
successCallback:(RCTResponseSenderBlock)successCallback
errorCallback:(RCTResponseSenderBlock)errorCallback) {

NSLog(@"Video constraint in RTCMediaStream get user media %@", constraints);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You can change it with RCTLog

@fukemy
Copy link

fukemy commented Nov 2, 2022

HI, can someone please help my problem: https://stackoverflow.com/questions/73554691/react-native-ios-error-build-undefined-symbol-objc-class-selfiesegmentator, I got this error and I can not run this project

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants