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

Add support for multiple frameBuffers #612

Merged
merged 1 commit into from
Nov 21, 2018

Conversation

tribal-tec
Copy link
Contributor

No description provided.

Copy link

@rdumusc rdumusc left a comment

Choose a reason for hiding this comment

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

Looks good to me

apps/ui/BaseWindow.cpp Outdated Show resolved Hide resolved
brayns/Brayns.cpp Outdated Show resolved Hide resolved
brayns/common/engine/Engine.cpp Outdated Show resolved Hide resolved
brayns/common/renderer/FrameBuffer.h Show resolved Hide resolved
rdumusc
rdumusc previously approved these changes Nov 20, 2018
Copy link
Contributor

@hernando hernando left a comment

Choose a reason for hiding this comment

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

It seems I'm already late, so I will post these comments and finish in another review.

apps/ui/BaseWindow.cpp Show resolved Hide resolved
apps/ui/BaseWindow.cpp Show resolved Hide resolved
apps/ui/BaseWindow.cpp Show resolved Hide resolved
apps/ui/BaseWindow.cpp Show resolved Hide resolved
brayns/Brayns.cpp Show resolved Hide resolved
brayns/Brayns.cpp Outdated Show resolved Hide resolved
brayns/common/engine/Engine.cpp Outdated Show resolved Hide resolved
brayns/common/engine/Engine.cpp Show resolved Hide resolved
brayns/common/renderer/FrameBuffer.h Outdated Show resolved Hide resolved
engines/ospray/OSPRayEngine.cpp Show resolved Hide resolved
Copy link
Contributor

@hernando hernando left a comment

Choose a reason for hiding this comment

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

It's interesting that the combined size of the left and right eye test images is quite bigger than the original side by side one.

brayns/Brayns.cpp Outdated Show resolved Hide resolved
plugins/Deflect/DeflectPlugin.cpp Outdated Show resolved Hide resolved
plugins/Deflect/DeflectPlugin.cpp Outdated Show resolved Hide resolved
plugins/Deflect/DeflectPlugin.cpp Outdated Show resolved Hide resolved
plugins/Rockets/SnapshotTask.h Outdated Show resolved Hide resolved
plugins/Rockets/RocketsPlugin.cpp Show resolved Hide resolved
plugins/Deflect/DeflectPlugin.cpp Show resolved Hide resolved
This is to support streaming stereo eye passes and deflect channels (Tide
surfaces) from plugins w/o having the core to know anything about that.
The frameBuffer names mean only something within the DeflectPlugin and other
plugins leveraging from that (e.g. OpenDeck).

Furthermore, this simplifies how stereo is handled, as this is just a
specializiaton of a multi buffer setup. For this, "--stereo" has been added
to the commandline args, which will create two buffers (one for each eye) at
startup. Side-by-side stereo output is implemented in braynsViewer, though the
viewer will output all buffers next to each other that have been added to
Brayns.
@tribal-tec
Copy link
Contributor Author

All should be good now

@tribal-tec tribal-tec merged commit 2c3c68a into BlueBrain:master Nov 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants