Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 MSAA option for Webgl2 #10052
Add MSAA option for Webgl2 #10052
Changes from 23 commits
fbbe3f3
f86c95b
59d7d2b
b9ac432
bd876c5
5532131
f56f3e4
4aa7b2e
818e24e
d8977d6
183782f
9209c99
614c88c
d22ee26
cbd8c45
a3ad889
b443feb
8c26a13
dd0a7f8
1301ddd
dd29f44
cd2b741
1bfb1f1
87997e2
895c641
0fa4811
c017f5a
67cdee4
b982262
fd84fd9
4c2d6f6
496b1c1
a2fa0e4
06190d3
2a04209
fbf3de9
ca82c10
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Should this be
format: pixelFormat
?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.
I don't think so - we have a validate function in
RenderbufferFormat
thatRenderbuffer
expects to pass in its constructor:cesium/Source/Renderer/RenderbufferFormat.js
Lines 14 to 23 in a11b14c
Though the RGBA8 format doesn't work with when the texture's datatype is floating point, which happens when HDR is turned on. Do we have a RenderbufferFormat that is expected to work with float textures?
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.
You might have to check https://www.khronos.org/registry/webgl/extensions/WEBGL_color_buffer_float/ to see if float textures are supported for renderbuffers.
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.
Interestingly, this Format Considerations section says that blitting from RGBA8 to RGBA32F (and I believe half float should work too) and vice versa should just work. Chrome and Firefox are reporting
src and dst formats differ for color
so I'm digging a bit deeper into this one.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.
You may need to convert the
pixelFormat
to the correctRenderBufferFormat
. Hardcoding the renderbuffer format to RGBA8 may break HDR.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.
I added a
RenderbufferFormat.getColorFormat
function in 895c641 that should do the trick. It feels a bit incorrect to pass aPixelDatatype
and compare it directly toWebGLConstants
values even if that works in this case; on the flip side, I didn't want to importPixelDatatype
inRenderbufferFormat
. Maybe thegetColorFormat
is better as a helper function in FramebufferManager.js?