-
Notifications
You must be signed in to change notification settings - Fork 163
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
Fix reload after a description with a mimic joint #212
Conversation
Thanks for opening this. For this PR, at the very least we should: a) target it to |
8e9dbb5
to
423d144
Compare
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
I've retargeted this to the |
Thanks for solving it Chris, and sorry for not following up (I was few days out). |
@Mergifyio backport iron humble |
✅ Backports have been created
|
* fix mimic_.clear(); * Make the copy explicit. * Add in a test for the mimic change crash. Signed-off-by: Chris Lalancette <clalancette@gmail.com> Co-authored-by: Guillaume Doisy <guillaume@dexory.com> (cherry picked from commit e72adfc) # Conflicts: # CMakeLists.txt
* fix mimic_.clear(); * Make the copy explicit. * Add in a test for the mimic change crash. Signed-off-by: Chris Lalancette <clalancette@gmail.com> Co-authored-by: Guillaume Doisy <guillaume@dexory.com> (cherry picked from commit e72adfc) # Conflicts: # CMakeLists.txt
* fix mimic_.clear(); * Make the copy explicit. * Add in a test for the mimic change crash. Signed-off-by: Chris Lalancette <clalancette@gmail.com> Co-authored-by: Guillaume Doisy <guillaume@dexory.com> (cherry picked from commit e72adfc)
* fix mimic_.clear(); * Make the copy explicit. * Add in a test for the mimic change crash. Signed-off-by: Chris Lalancette <clalancette@gmail.com> Co-authored-by: Guillaume Doisy <guillaume@dexory.com> (cherry picked from commit e72adfc)
* fix mimic_.clear(); * Make the copy explicit. * Add in a test for the mimic change crash. Signed-off-by: Chris Lalancette <clalancette@gmail.com> Co-authored-by: Guillaume Doisy <guillaume@dexory.com> (cherry picked from commit e72adfc) Co-authored-by: Guillaume Doisy <doisyg@users.noreply.github.com>
* fix mimic_.clear(); * Make the copy explicit. * Add in a test for the mimic change crash. Signed-off-by: Chris Lalancette <clalancette@gmail.com> Co-authored-by: Guillaume Doisy <guillaume@dexory.com> (cherry picked from commit e72adfc) Co-authored-by: Guillaume Doisy <doisyg@users.noreply.github.com>
Works for me to fix: #211
Not sure it is the cleanest approach through, opening as a draft to discuss first.