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

4.5.0 j #3803

Merged
merged 18 commits into from
Oct 30, 2024
Merged

4.5.0 j #3803

merged 18 commits into from
Oct 30, 2024

Conversation

jinyuagora
Copy link
Collaborator

No description provided.

<!DOCTYPE reference PUBLIC "-//OASIS//DTD DITA Reference//EN" "reference.dtd">
<reference id="api_irtcengine_setexternalremoteeglcontext">
<title><ph keyref="setExternalRemoteEglContext"/></title>
<shortdesc id="short"><ph id="shortdesc">设置远端视频流渲染的 EGL 环境上下文。</ph></shortdesc>
Copy link
Collaborator

Choose a reason for hiding this comment

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

“远端视频流渲染”该如何理解?意思是说自采集的外部视频流在本地渲染时的 EGL 上下文么?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

这个方法和自采集其实没有关系,主要是自渲染

我理解指的是:在本地渲染从远端接收的视频流时,所使用的 EGL 上下文

Copy link
Collaborator

Choose a reason for hiding this comment

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

哦那为了避免歧义,是不是可以改为:设置远端视频流在本地渲染时的 xxx 上下文。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

这个要么保持原状吧,两次评审会上没人提问题,刚和开发确认了下他回复也不是很明朗。

dita/RTC-NG/API/class_externalvideoframe.dita Outdated Show resolved Hide resolved
dita/RTC-NG/API/class_videoframe.dita Outdated Show resolved Hide resolved
private long timestampNs;
private ColorSpace colorSpace;
private SourceType sourceType;
private float sampleAspectRatio;

private AlphaStitchMode alphaStitchMode = AlphaStitchMode.ALPHA_NO_STITCH;
Copy link
Collaborator

Choose a reason for hiding this comment

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

原型改了,alphaStitchMode 增加了枚举类,不再是 int 了,帮忙在下面成员参数那里改一下呗

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

改了但没完全改,貌似只有这个VideoFrame的原型里改了

枚举类原型定义里没有:
image
AgoraVideoFrame里也还是int:
image

你看要不要确认下先

@jinyuagora jinyuagora merged commit 8aff75a into release/rtc-ng/4.5.0 Oct 30, 2024
2 checks passed
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