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

Fixed Sponza loading in EMSCRIPTEN #21

Merged
merged 2 commits into from
Nov 16, 2024
Merged

Fixed Sponza loading in EMSCRIPTEN #21

merged 2 commits into from
Nov 16, 2024

Conversation

maniatic0
Copy link
Contributor

@maniatic0 maniatic0 commented Nov 15, 2024

I added a fix to load Sponza in the browser. It embeds the binary into the wasm and loads it in the virtual filesystem. For C++ all of this is invisible..

tiny_bvh version 0.8.3 performance statistics (emcc 3.1 build)
----------------------------------------------------------------
Loading triangle data (262267 tris).
BVH construction speed
warming caches...
- reference builder:  463.07ms for  262267 triangles - 501326 nodes, SAH=214.22
- fast AVX builder:   211.15ms for  262267 triangles - 500482 nodes, SAH=212.78
BVH traversal speed
- CPU, coherent,   basic 2-way layout, ST:   4482.8ms for   7.68M rays =>   1.71MRay/s
- CPU, coherent,   alt 2-way layout,   ST:   4105.7ms for   7.68M rays =>   1.87MRay/s
- CPU, coherent,   soa 2-way layout,   ST:   3937.2ms for   7.68M rays =>   1.95MRay/s
- CPU, coherent,   basic 2-way layout, MT:    837.0ms for   7.68M rays =>   9.18MRay/s
- CPU, coherent,   2-way, packets,     MT:    193.5ms for   7.68M rays =>  39.68MRay/s
- CPU, coherent,   2-way, packets/SSE, MT:    144.8ms for   7.68M rays =>  53.04MRay/s
Optimizing BVH, regular...   done (12.34s). New: 501325 nodes, SAH=181.85
- CPU, coherent,   2-way optimized,    ST:   3047.6ms for   7.68M rays =>   2.52MRay/s
- CPU, incoherent, basic 2-way layout, MT:   1011.5ms for   7.68M rays =>   7.59MRay/s

@jbikker jbikker merged commit 8b11e63 into jbikker:main Nov 16, 2024
6 checks passed
@jbikker
Copy link
Owner

jbikker commented Nov 16, 2024

Thanks, merged!

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