-
-
Notifications
You must be signed in to change notification settings - Fork 313
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
refactor: port layer to Cobra renderer: Noise gradient #3306
base: master
Are you sure you want to change the base?
Conversation
Does supersample parameter any effect in original implementation? v1.0.2 or 1.4.x or 1.5.1 ... |
That's strange. It did some calculations, but I didn't see any difference in the image.
|
Neither did I... |
And does the PR implementation work equally? |
No, toggling the parameter has no effect. |
e6262cc
to
b50f901
Compare
I believe now I reproduce the supersample effect. :) @Svarov-RZM @ice0 |
Yeah, I confirm. It's working properly now. |
@Svarov-RZM well, I try to port as long as I understand the engine and its code. :) |
Don't worry, it will perform better, since algorithm stays the same but the unnecessary wrapper is removed. Even if, by some convoluted reason, it won't, the performance can be addressed later after porting. What's more important is stability. Currently layers such as Twirl, Curve Warp and Spherize are close to being unusable. They crash on zoom in (so annoying!) and often hang Synfig if you use it on complex work. But anyway, there's no hurry, when something is ready we'll test it and fix whatever issues come along. |
No description provided.