search

buddhi1980 / mandelbulber2

Official repository for Mandelbulber v2
GNU General Public License v3.0
893 stars 116 forks source link

revisit case analyticFunctionLogarithmic: #917

Closed mclarekin closed 2 years ago

mclarekin commented 2 years ago

This is a reminder to look further into this. For now i will just add to this issue as i remember where if (extendedAux.r > 1.0) causes some problems This has not been updated for hybrid mode. THIS IS GOOD FOR NOW as it allows easy testing

                case analyticFunctionLogarithmic:
                {
                    if (extendedAux.r > 1.0)
                        out->distance = 0.5 * r * log(r) / extendedAux.DE;
                    else
                        out->distance = 0.0;
                    break;
                }
buddhi1980 commented 2 years ago

Thanks for showing me this. I forgot u update it for hybrids

mclarekin commented 2 years ago

https://github.com/buddhi1980/mandelbulber2/commit/248aa5dfb2bfe64ceb6d758aaa1793abd7b2324c

fixed it as i cannot currently remember the problem i wanted to test