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.

Sign up for GitHub

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

Jump to bottom

Asynchronous shader compilation #1209

Merged
merged 24 commits into from
Mar 8, 2023
Merged

Asynchronous shader compilation #1209

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
d7e1d16
Preliminary testing code.
Feb 24, 2023
143164a
Code for measuring render loop times.
Feb 24, 2023
6cad4f6
Fix test code.
Feb 24, 2023
8460274
Refactor experience.js.
Feb 28, 2023
38c273a
Preliminary code to separate out shader compilation from CreateProgram.
Feb 28, 2023
9257652
Force shader compilation in the test code.
Mar 1, 2023
1f04b1c
Make async code work + rename function.
Mar 1, 2023
ec25c68
Separate synchronous and asynchronous versions of CreateProgram.
Mar 2, 2023
f814deb
Assign the program handle on the Javascript thread.
Mar 3, 2023
77f5c19
Restore original experience.js.
Mar 3, 2023
1df96a2
Return both the program handle and VertexAttributeLocations from Crea…
Mar 4, 2023
209c34a
Throw a proper error after encountering an async shader compilation e…
Mar 6, 2023
d4401b7
Simplify error reporting.
Mar 6, 2023
678fb49
Preliminary code for movable ProgramData (not working yet).
Mar 7, 2023
5e66231
Use unique_ptr to make the Arcana return type work.
Mar 7, 2023
eb5a104
Initialize member variables in constructor.
Mar 8, 2023
f8c062d
Remove unnecessary release of unique_ptr.
Mar 8, 2023
089a278
Leave parameter of nulled operator anonymous.
Mar 8, 2023
e1872d5
Indent lambda captures.
Mar 8, 2023
2319953
Remove unused lambda capture.
Mar 8, 2023
654886c
Remove ProgramData.Disposed.
Mar 8, 2023
8f17fb7
Use pattern similar to CreateProgramAsync in CreateProgram.
Mar 8, 2023
4977f3e
Use BGFX_INVALID_HANDLE instead of bgfx::kInvalidHandle.
Mar 8, 2023
17666c0
Use brace initialization for BGFX handles.
Mar 8, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions Plugins/NativeEngine/Source/NativeEngine.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -729,7 +729,7 @@ namespace Babylon
ProgramData* program = new ProgramData{};
bghgary marked this conversation as resolved.
Show resolved Hide resolved
try
{
*program = std::move(*CreateProgramInternal(vertexSource, fragmentSource).release());
*program = std::move(*CreateProgramInternal(vertexSource, fragmentSource));
}
catch (const std::exception& ex)
{
Expand All @@ -750,16 +750,16 @@ namespace Babylon
Napi::Value jsProgram = Napi::Pointer<ProgramData>::Create(info.Env(), program, Napi::NapiPointerDeleter(program));

arcana::make_task(arcana::threadpool_scheduler, *m_cancellationSource,
[this, vertexSource, fragmentSource, program, cancellationSource{m_cancellationSource}]() -> std::unique_ptr<ProgramData>
[this, vertexSource, fragmentSource, cancellationSource{m_cancellationSource}]() -> std::unique_ptr<ProgramData>
{
return CreateProgramInternal(vertexSource, fragmentSource);
})
.then(m_runtimeScheduler, *m_cancellationSource,
[program,
jsProgramRef{Napi::Persistent(jsProgram)},
onSuccessRef{Napi::Persistent(onSuccess)},
onErrorRef{Napi::Persistent(onError)},
cancellationSource{m_cancellationSource}](const arcana::expected<std::unique_ptr<ProgramData>, std::exception_ptr>& result)
jsProgramRef{Napi::Persistent(jsProgram)},
onSuccessRef{Napi::Persistent(onSuccess)},
onErrorRef{Napi::Persistent(onError)},
cancellationSource{m_cancellationSource}](const arcana::expected<std::unique_ptr<ProgramData>, std::exception_ptr>& result)
{
if (result.has_error())
{
Expand Down
14 changes: 9 additions & 5 deletions Plugins/NativeEngine/Source/NativeEngine.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,18 +44,22 @@ namespace Babylon
{
ProgramData() = default;
ProgramData(const ProgramData&) = delete;
ProgramData& operator=(const ProgramData& other) = delete;

ProgramData(ProgramData&& other) noexcept
ProgramData& operator=(const ProgramData&) = delete;

ProgramData(ProgramData&& other) noexcept:
Handle{other.Handle},
Uniforms{std::move(other.Uniforms)},
UniformNameToIndex{std::move(other.UniformNameToIndex)},
UniformInfos{std::move(other.UniformInfos)},
VertexAttributeLocations{std::move(other.VertexAttributeLocations)}
{
(*this) = std::move(other);
other.Handle = bgfx::ProgramHandle{bgfx::kInvalidHandle};
bghgary marked this conversation as resolved.
Show resolved Hide resolved
}

ProgramData& operator=(ProgramData&& other) noexcept
{
Handle = std::move(other.Handle);
other.Handle = bgfx::ProgramHandle{bgfx::kInvalidHandle};
bghgary marked this conversation as resolved.
Show resolved Hide resolved
Disposed = std::move(other.Disposed);
Uniforms = std::move(other.Uniforms);
UniformNameToIndex = std::move(other.UniformNameToIndex);
UniformInfos = std::move(other.UniformInfos);
Expand Down