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

So the problem is that when your inside the sphere it won't render so it... #895

Closed
wants to merge 1 commit into
base: development
from

Conversation

Projects
None yet
2 participants
@Winterleaf
Contributor

Winterleaf commented Nov 4, 2014

... might make someone

think that it's not working right. So what I did was determine if the camera is inside the sphere.
If the camera is inside the sphere, then I find the distance from the center of the sphere to the camera
Round down and use that as the radius to draw the sphere.
That way if someone zooms in or out, their screen is still showing the sphere.

@Winterleaf Winterleaf closed this Nov 4, 2014

So the problem is that when your inside the sphere it won't render so…
… it might make someone

think that it's not working right.  So what I did was determine if the camera is inside the sphere.
If the camera is inside the sphere, then I find the distance from the center of the sphere to the camera
Round down and use that as the radius to draw the sphere.
That way if someone zooms in or out, their screen is still showing the sphere.

@crabmusket crabmusket added the Defect label Nov 4, 2014

@crabmusket crabmusket added this to the 3.7 milestone Nov 4, 2014

@crabmusket

This comment has been minimized.

Show comment
Hide comment
@crabmusket

crabmusket Nov 4, 2014

Contributor

Please remove the WLE-Vince comment block. git blame will show the history and we like to keep comments free of personal details going forwards :).

Oh, didn't realised you closed it. Will repeat my comment on the new issue.

Contributor

crabmusket commented Nov 4, 2014

Please remove the WLE-Vince comment block. git blame will show the history and we like to keep comments free of personal details going forwards :).

Oh, didn't realised you closed it. Will repeat my comment on the new issue.

@crabmusket crabmusket referenced this pull request Jan 25, 2015

Merged

Forest wind emitter #1124

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment