[need testing] New opacity handling in the brushes needs testing by users

Hi, all!

I’ve been recently working on an ancient opacity calculation bug from Krita 2.9.x era, and I have finally fixed it! :slight_smile: The problem is that it changes the way how we handle opacity on a very low level, so I would really like someone to test the actual code before I merge it into master! :slight_smile:

Test plan:

  1. Paint with different brushes with different non-100% opacity and flow levels. They should behave as usual :slight_smile:
    • with normal Pixel bursh (non-standard opacity and flow)
      • Build-up mode
      • Incremental mode
    • with Color Smudge brush (non-standard opacity):
      • new algorithm
      • old algorithm
      • lightness-mode
  2. Fill Tool with custom opacity value
  3. Enclosed fill tool with custom opacity value (@Deif_Lou, ping :slight_smile: )

Basically, everything should work “as usual” :slight_smile: The main worry for me is the colorsmudge brush. Theoretically, the patch changes its algorithm a little bit. And again, theoretically, it should become a little bit smooth and precise. But there might be some issues that I don’t see while testing.

Could you please briefly check the packages and see if there any obvious regressions in the opacity handling?

5 Likes

I’m not sure about incremental but if that is “wash” then that and build up mode with pixel brushes behave the same as the old algorithm.

I also used the wet bristles brush on normal and lightness and those also behave the same as old.

Will try to get to the other test cases later.

I tried with a 16 bit sRGB document, Windows, Krita 5.2.6, 5.3 alpha from above linked source.

with normal Pixel bursh (non-standard opacity and flow)

  • Build-up mode
  • Incremental mode (I assume this is meant to be wash mode?)

with Color Smudge brush (Blender Blur from Krita3 resources):

  • lightness-mode (from HSL blending modes)
  • normal-mode
    (I don’t know about new / old algorithm)

Fill Tool Bucket.

I did not notice a relevant difference in how drawing, blending, wash / build up, fill behave.


some patches are made 5.3 some in 5.2.6

1 Like

Tested here and everything looks alright

1 Like

Hi, @Reinold, @cgidesign and @Zoldyako!

Thank you very much for testing! I will merge the patch then! :slight_smile:

4 Likes

This topic was automatically closed 4 days after the last reply. New replies are no longer allowed.