Skip to content

Conversation

@AndrewRayCode
Copy link
Contributor

Addresses issue #14. Attempted to follow current conventions but it is
likely users will want more control over rotation.

I don't think this is quite ready because I see corners appearing on the rotated particles. need to investigate the 2d sampler. Possibly it's because I have particle texture tiling turned on and just need to turn it off.

Addresses issue squarefeet#14. Attempted to follow current conventions but it is
likely users will want more control over rotation.
@squarefeet
Copy link
Owner

Nice one, dude. I'll take a look over this within the next few days. At first glance it looks all good - thanks for sticking to the style :)

Stemkoski was experimenting with aligning a particle's angle to its velocity - all of that stuff is in the 'dev' branch if you want to take a peek. His original pull req is here.

What I'll probably do is merge this into a new branch to make sure it's all good, then merge into master and bump the version number (adding credit, of course). Then once Stemkoski and I have the time to finish up the angle-aligning stuff (it's very close!), I'll merge the dev branch. Sound good?

@squarefeet
Copy link
Owner

PS. Regarding the corners on particles - it does sound like it's a tex tiling issue. Let me know how you get on.

@AndrewRayCode
Copy link
Contributor Author

It was indeed because of the tiling.

@AndrewRayCode
Copy link
Contributor Author

I get unpredicted results on the dev branch with a emitter type disk. my particles shoot out even though speed is zero.

@squarefeet
Copy link
Owner

Yeah, that'll be the gravity stuff I was playing around with. It is a dev
branch, please don't assume it to be stable!

On 9 January 2014 06:24, Andrew Ray notifications@github.com wrote:

I get unpredicted results on the dev branch with a emitter type disk. my
particles shoot out even though speed is zero.


Reply to this email directly or view it on GitHubhttps://github.com//pull/16#issuecomment-31905837
.

@AndrewRayCode
Copy link
Contributor Author

Ah. In the future, unfinished code should be in its own branch, and merged to dev when it is 100% working. Otherwise other people branching off dev for development could get hosed by broken code.

@squarefeet
Copy link
Owner

Noted.

@squarefeet squarefeet closed this Jan 13, 2014
AndrewRayCode added a commit to AndrewRayCode/ShaderParticleEngine that referenced this pull request Jan 22, 2014
Addresses issue squarefeet#16. Take two, this time off of dev taking into account
stemkoski's changes.

Updates readme to include angleAlignVelocity
AndrewRayCode added a commit to AndrewRayCode/ShaderParticleEngine that referenced this pull request Jan 24, 2014
Addresses issue squarefeet#16. Take two, this time off of dev taking into account
stemkoski's changes.

Updates readme to include angleAlignVelocity
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