Skip to content

Update Mandelbrot.pde #765

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

Merged
merged 4 commits into from
Jul 15, 2019
Merged

Update Mandelbrot.pde #765

merged 4 commits into from
Jul 15, 2019

Conversation

l8l
Copy link
Contributor

@l8l l8l commented Jul 8, 2019

With the proposed changes, the plot of the Mandelbrot set will appear smoother than before. This is achieved by measuring how much the maximum threshold was exceeded by the final iteration of the Mandelbrot function z -> z^2 + z_0 at each point z_0.

With the proposed changes, the plot of the Mandelbrot set will be smoother than before. This is achieved by measuring how much the maximum threshold was exceeded.
Copy link
Member

@shiffman shiffman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few small comments regarding using float instead of double, if they are ok with you, you can click "accept suggestion".

@l8l
Copy link
Contributor Author

l8l commented Jul 12, 2019

Thanks @shiffman . In this case, we only need float because the precision needed is small? If we'd implement a Mandelbrot with zoom function, would we need to use double instead?

@shiffman
Copy link
Member

@l8l double would almost certainly be preferred for calculations like these and you are right I think that a higher degree of precision would be required when zoomed in! But for a basic Processing example like this one, I prefer to stick with float as "good enough" for ease / simplicity!

@shiffman
Copy link
Member

Thank you for this contribution!! ❤️

@shiffman shiffman merged commit 53f7959 into processing:master Jul 15, 2019
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