-
Notifications
You must be signed in to change notification settings - Fork 108
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
Fractal noise scaling #333
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When computing a fractal noise, the sum of all octaves is normalized by dividing it by the sum of the amplitudes of the octaves. Here's the code of the function which computes the normalizing factor
amp_fractal
:The computed sum
amp_fractal
has one more term than the number of octaves. I believe this is wrong. For example, in the particular case where you have one octave only, it ends up rescaling the noise by dividing it by (1 + gain) as if we had two octaves when it should divide by 1 instead (i.e. no normalization needed since there's only one octave). This can be confirmed by looking at Auburn's original code.The text was updated successfully, but these errors were encountered: