Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rodolforg
Copy link
Contributor

No description provided.

@rodolforg
Copy link
Contributor Author

Does supersample parameter any effect in original implementation? v1.0.2 or 1.4.x or 1.5.1 ...
Here it has none...

@ice0
Copy link
Collaborator

ice0 commented Jan 18, 2024

Does supersample parameter any effect in original implementation? v1.0.2 or 1.4.x or 1.5.1 ...
Here it has none...

That's strange. It did some calculations, but I didn't see any difference in the image.

if(super_sample&&pixel_size)
{
...
}

@rodolforg
Copy link
Contributor Author

Neither did I...

@Svarov-RZM
Copy link
Contributor

Does supersample parameter any effect in original implementation?

It does in all the versions. You just need to create a gradient where such effect would be visible, like in this case:
NG-SuperSample

On the right side the gradient was generated with "Super Sampling" enabled and you can clearly see it's anti-aliased.

Do I personally think this parameter is needed? No. We already have a dedicated layer for this purpose called "Super Sample" which provides the user with more options for anti-aliasing.

@rodolforg
Copy link
Contributor Author

And does the PR implementation work equally?

@Svarov-RZM
Copy link
Contributor

And does the PR implementation work equally?

No, toggling the parameter has no effect.
Here's a test file:
NG-SuperSample.zip

@rodolforg
Copy link
Contributor Author

rodolforg commented Jan 21, 2024

I believe now I reproduce the supersample effect. :) @Svarov-RZM @ice0

@Svarov-RZM
Copy link
Contributor

I believe now I reproduce the supersample effect. :)

Yeah, I confirm. It's working properly now.
Thank you for continuing on porting layers to the new engine, really appreciate it.
If you need any testing, just ping me.

@rodolforg
Copy link
Contributor Author

@Svarov-RZM well, I try to port as long as I understand the engine and its code. :)
I don't check if it performed better, though.

@Svarov-RZM
Copy link
Contributor

I don't check if it performed better, though.

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.

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.

None yet

3 participants