Maniphest T83657

Pose Mode: Clearing transform doesn't respect Mirror X
Closed, ResolvedKNOWN ISSUE

Assigned To
Sybren A. Stüvel (sybren)
Authored By
Luciano Muñoz Sessarego (looch)
Dec 11 2020, 2:17 PM
Tags
  • BF Blender
  • Animation & Rigging
Subscribers
Erick Tukuniata (erickblender)
Germano Cavalcante (mano-wii)
Karthik Rangasai Sivaraman (rangasai)
Luciano Muñoz Sessarego (looch)
Philipp Oeser (lichtwerk)
Sybren A. Stüvel (sybren)
Tokens
"100" token, awarded by Juangra_Membata.

Description

System Information
Operating system: Windows-10-10.0.18362-SP0 64 Bits
Graphics card: GeForce RTX 2070 with Max-Q Design/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 457.51

Blender Version
Broken: version: 2.92.0 Alpha, branch: master, commit date: 2020-12-09 01:29, hash: rB9b11a7776f2a
Worked: (newest version of Blender that worked as expected)

Short description of error
When animating and posing with X mirror. One performs many actions and some of them don't transfer to the other side and they should, simple operations like: Resetting Transforms, Breakdowner,

Exact steps for others to reproduce the error
open rain rig from the cloud.
turn on x mirror
move a hand
both move.
now press alt r / g / s to get it back to its place,
the other hand doesnt respond.

Revisions and Commits

rB Blender
D9938
D9938

Event Timeline

Luciano Muñoz Sessarego (looch) created this task.Dec 11 2020, 2:17 PM
Luciano Muñoz Sessarego (looch) added a project: Animation & Rigging.
Philipp Oeser (lichtwerk) changed the task status from Needs Triage to Confirmed.Dec 11 2020, 4:37 PM
Philipp Oeser (lichtwerk) added subscribers: Sybren A. Stüvel (sybren), Philipp Oeser (lichtwerk).

Can confirm the behavior.

Will leave up to @Sybren A. Stüvel (sybren) to decide if we want to keep (since this is the grey zone between request/limitation/bug).
I'd say X-Mirror is not garuanteed to work across all transform related tools [it would certainly be nice and consistent].

Sybren A. Stüvel (sybren) changed the subtype of this task from "Report" to "Known Issue".Dec 18 2020, 11:22 AM
Sybren A. Stüvel (sybren) removed a subscriber: Sybren A. Stüvel (sybren).

I'll tag this as a Known Issue, and I've also added it to the Weak Areas wiki page.

Sybren A. Stüvel (sybren) moved this task from Backlog to Known Issues on the Animation & Rigging board.Dec 18 2020, 11:22 AM
Luciano Muñoz Sessarego (looch) added a subscriber: Sybren A. Stüvel (sybren).EditedDec 18 2020, 11:51 AM

@Sybren A. Stüvel (sybren) shouldnt we discuss it a bit before moving it to known issues?, maybe improving it isn't a great deal but it would bring huge benefit workflow wise.

Sybren A. Stüvel (sybren) added a comment.Dec 18 2020, 1:08 PM

The Known Issues category isn't a death sentence ;-) We can continue discussing, and people can still pick it up to work on it. Sometimes Known Issues are more fun to work on than bug reports, even!

Karthik Rangasai Sivaraman (rangasai) added a subscriber: Karthik Rangasai Sivaraman (rangasai).Dec 24 2020, 5:10 PM

Hi all, I'm new to blender development. I have downloaded and built blender on my system and I could successfully reproduce the error as well. I would like to work on this issue as my first one. Will that be fine?
If yes, could you please guide me in the right direction?

Garry R. Osgood (grosgood) added a subscriber: Garry R. Osgood (grosgood).Dec 24 2020, 6:25 PM

@Karthik Rangasai Sivaraman (rangasai)
Welcome!
When you've developed a patch that addresses the issue, create a differential. See Contributing Code for guidelines. To get a sense of who your reviewers should be, look up the file(s) your patch will change in Diffusion and study the colorful left hand column which shows which commit has led to the lines of code on the right. The committers' avatars are a part of that - they constitute potential reviewers. Doesn't hurt to read up on the neighboring commits around the regions you are patching - could be gotchas there that you need to familiarize yourself with. git log -L is also your friend. Have fun!

Karthik Rangasai Sivaraman (rangasai) added a comment.Dec 24 2020, 6:46 PM

@Garry R. Osgood (grosgood)
Thank you for the advice. This is helpful. I will follow them.

Garry R. Osgood (grosgood) removed a subscriber: Garry R. Osgood (grosgood).Dec 24 2020, 7:08 PM
Karthik Rangasai Sivaraman (rangasai) added a comment.Dec 28 2020, 8:18 AM
This comment was removed by Karthik Rangasai Sivaraman (rangasai).
Juan Gabriel (Juangra_Membata) awarded a token.Dec 28 2020, 9:15 AM
Sybren A. Stüvel (sybren) renamed this task from In pose mode: Mirror X doesnt work with certain operations to Pose Mode: Clearing transform doesn't respect Mirror X.Jan 18 2021, 11:57 AM
Sybren A. Stüvel (sybren) closed this task as Resolved by committing rBf7a5695676dc: Fix T83657: Pose Mode: Clearing transform doesn't respect Mirror X.Jan 18 2021, 11:59 AM
Sybren A. Stüvel (sybren) claimed this task.
Sybren A. Stüvel (sybren) added a commit: rBf7a5695676dc: Fix T83657: Pose Mode: Clearing transform doesn't respect Mirror X.
Sybren A. Stüvel (sybren) added a commit: rBc85317e66940: Fix T83657: Pose Mode: Clearing transform doesn't respect Mirror X.Jan 18 2021, 12:03 PM
Germano Cavalcante (mano-wii) added subscribers: Erick Tukuniata (erickblender), Germano Cavalcante (mano-wii).