-
Notifications
You must be signed in to change notification settings - Fork 396
[FEAT]Multiple video capturer #73
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
Conversation
Change the access modifiers from public to internal or protected.
add AudioStreamTrack.
public modify to internal.
# Conflicts: # Assets/Scripts/RenderStreaming.cs
…ream CaptureAudioStream
todo: 1.create multiple resolution texture for nvEncoder. 2.Abstract out a base class BaseEncoder. Derive AmdEncoder and softEncoder.
|
Currently, |
|
Could you add auto testing to check multiple video tracks? |
|
I will test this PR on my PC |
| } | ||
|
|
||
| void NvEncoder::InitEncoder(int width, int height) | ||
| void NvEncoder::InitEncoder(int width, int height, int _bitRate) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you keep the naming rules?
_bitRate to bitRate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this.bitRate = bitRate;
decoupling video stream(audio stream) and video track(audio track). all capturers share one encoder.