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

Feature/issue 71 gpu renderer #89

Merged
merged 40 commits into from
Feb 4, 2020
Merged

Feature/issue 71 gpu renderer #89

merged 40 commits into from
Feb 4, 2020

Conversation

rohan-deshpande
Copy link
Collaborator

Added

  • src/renderers/GPURenderer module which is an experimental new renderer that aims to improve the performance of sprite based particle systems
  • sandbox directory for quickly debugging new features/experiments without too much "boilerplate" overhead
  • sandbox script for booting the sandbox and wiring in auto updates to the src into it

rohan-deshpande and others added 30 commits January 14, 2020 17:30
@rohan-deshpande
Removed rule to error on incorrect line break style
7223411
@rohan-deshpande
Added routes and stubbed examples for new renderers
725ce73
@rohan-deshpande
Removed ./ from link-src script which seems to break on Windows
cfc4c5a
@rohan-deshpande
Added simple sandbox for windows dev
4f5ecdb
@rohan-deshpande
Cleaned up the commands a bit
ad83c96
@rohan-deshpande
Now gitignoring lib sent to sandbox
4a23134
@rohan-deshpande
Added specific webpack config for sandbox because windows doesn't lik…
47977b3 
…e inline env vars
@manthrax
Initial grafting of Buffered Particles.
b8fa42d
@rohan-deshpande
Added stats-js as dep to sandbox via unpkg cdn, also added the API in…
8e2194f 
…tegration in the vis class
@manthrax
Add feature flag for buffered particles..
a2b09f9 
Crank particle count for stress test..
@manthrax
Merge branch 'feature/issue-71-new-renderers' of https://github.com/c…
02fbe7c 
…reativelifeform/three-nebula into feature/issue-71-new-renderers
@manthrax
Detach particle container from rendering for buffered.
08b31f7
@rohan-deshpande
Added points renderer experiment
c09e7dc
@rohan-deshpande
Wasn't adding the points object to the scene, still not fixed but at …
a786c8e 
…least it is rendering something... sometimes
@rohan-deshpande
Page title
dde1221
@rohan-deshpande
Added prototype ParticleBuffer class
8a58bb5
@rohan-deshpande
Created the base for the new PointsRenderer, got it rendering somethi…
ee09fa2 
…ng but still far from anything usable
@rohan-deshpande
Getting deeper into the mystery, particles are being killed before th…
c15aeb6 
…ey've finished their life cycle
@rohan-deshpande
ITS WORKING
ef76787
@rohan-deshpande
Have moved to using a single interleaved buffer. Point position is wo…
79c96eb 
…rking but I think I need shaders for size, colour and alpha
@rohan-deshpande
Color now working in the shaders
a7ca872
@rohan-deshpande
Alpha also working, that wasn't hard
e4f90ce
@rohan-deshpande
Textures now working, we are done diddily done neighbourino
f9b078e
@rohan-deshpande
Of course there's a bug. Eightdiagrams is leaving artifacts for some …
d0a97ea 
…reason
@rohan-deshpande
Fixed the bug, added gravity example to sandbox, multiple emitters no…
0941f8f 
…w work
@rohan-deshpande
Changed list to uniqueList
1b4c121
@rohan-deshpande
Added max number of items prop to UniqueList
cb37d43
@rohan-deshpande
Updated sandbox visualisation and run script to have the potential fo…
5e9b8b2 
…r maxTicks
@rohan-deshpande
Added GPURenderer submodule to renderers
84e58f0
@rohan-deshpande
Something broke when I ported it into src, need to debug in the sandbox
b7b3125
@rohan-deshpande
Debugged it, two THREEs were causing problems. The PointZone example …
39e03f9 
…isn't working though, need to debug this
@rohan-deshpande
Added orbit controls to sandbox and confirmed point zone example is b…
a47ec50 
…roken
@rohan-deshpande
Attempted to implement a pointId on particles, didn't work, uniqueLis…
aaeebfb 
…t won't ever really get too big but maybe need to pool here as well
@rohan-deshpande
Fixed it, the problem was that without a texture to render, the frag …
a36747a 
…shader just rendered nothing
@rohan-deshpande
Added better GPURenderer example
44e8ab6
@rohan-deshpande
Added coverage for emitter.index and pool get/create args
ce0e05a
@rohan-deshpande
Removed GPURenderer from code coverage report for now as it's experim…
c87b18c 
…ental, added a note in doc block for this
@rohan-deshpande
Fixed bundlesize issue
c2f2c61
@rohan-deshpande
Added new check-bundlesize script to isolate the bundlesize checking …
efb3798 
…flow from needing to actually bundle the lib, added this to travis.yml
@rohan-deshpande
Reverting build to develop branch, will build there
d20a934
@rohan-deshpande rohan-deshpande merged commit 08914fc into develop Feb 4, 2020
@rohan-deshpande rohan-deshpande deleted the feature/issue-71-gpu-renderer branch February 4, 2020 08:43
@rohan-deshpande rohan-deshpande mentioned this pull request Feb 4, 2020
Closed
@rohan-deshpande rohan-deshpande mentioned this pull request Jun 2, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rohan-deshpande @manthrax