To me this looks fine.
The actual placement of buttons and decision on the consistency in the clip editor I'd leave to Pablo.

Thanks for the future improvement! This is a great thing
that will help many animators, using only the Video Sequencer
and a regular image, which will make it possible to better animate
selected areas using a mask, turning a static image into a dynamic one.

Looking at this, solution seems to be simple, but there may be problems implementing this without some major refactoring being committed first.
I have patches for this change prepared, but it probably won't happen in current release, only in 2.93.

Indeed the same as T84202: Sculpt crash after remesh (mask operators, cloth brush, ...), will merge.

Don't worry.... Someone from the staff will probably merge this task with the other one soon enough...

@TheRedWaxPolice (TheRedWaxPolice),
Though the other report uses the lasso...
this looks like the same issue to me.

Here.. T84202: Sculpt crash after remesh (mask operators, cloth brush, ...)

Will classify as bug for now.

Changing the handle type seems to play an important part, animation of just the point positions seems to be fine.

I have this on the list for tomorrow to look at again

@Philipp Oeser (lichtwerk) mask surely shouldn't be mangled after modifying it.
Could be missing DEG_id_update_tag call. How do i reproduce the issue with the attached file?

Here is a file with mask consisting only of 4 point.

From a quick look, the curve is flipped, switching direction makes it work. Even so, this should still be supported.

I thought T69079: KeyingScreen Compositor node emits "black holes" was related (but seems like this is a different way rasterizing/filling/).

Think it's fine. There are other special cases in special_aftertrans_update which does DEG tag as well, so seems this is indeed how it's supposed to be.

@Sergey Sharybin (sergey): anything more to check?

Changing back to Report from Bug until we know if this is supposed to be supported...

In D7786#187755, @Sergey Sharybin (sergey) wrote:

Think its fine but a bit weird to do DEG_id_tag_update since the mask was transformed, so should have been tagged somewhere for that already?

Oh! sorry about that- the video is public now, though it sounds like you've already seen what it shows.

Think its fine but a bit weird to do DEG_id_tag_update since the mask was transformed, so should have been tagged somewhere for that already?

In T76872#934599, @Ian Hubert (mrdodobird) wrote:

AH! Cool hey, thanks, Philipp!

So- I thought I had my head wrapped around this (this is my first bug, so I checked everything like crazy), but I found a couple things out.

1.) it doesn't have anything to do with auto-keyframing. After you add a mask with keyframes, the next mask will get the weird warbles.
2.) I can avoid the issue if I create a new mask layer and add the mask to that, (which is honestly a pretty reasonable workaround for most instances)

But also- if I'm understanding your suggestion right, I've done a few tests with it, in every order I can think of, and it doesn't seem like changing the selection makes any difference.

Youtube video seems to be private.

AH! Cool hey, thanks, Philipp!

Confirming on first sight.