search

taiya / dgp

Digital Geometry Processing - Fall 2016 - University of Victoria
38 stars 11 forks source link

Question: Laplacian weight matrix #25

Closed xuzheng0927 closed 7 years ago

xuzheng0927 commented 7 years ago

In hw3 part 2, I tried using Laplacian-Beltrami weight matrix WITHOUT 1/(2*A_i) coefficients, and it gave me the correct output (the original function in the code base uses the area coefficients, with which I always got exploding points).

I also noticed that in ARAP, the weight is without 1/(2*A_i) as well. Should that be removed?

taiya commented 7 years ago

1) biharmonic deformation: you are using a SPD (symmetric positive definite) factorizer. Is your system matrix SPD? (potentially not) The slides (fairing) and the book's appendix cover how to correct this. 2) ARAP: carefully read the end of Sec. 2 in the paper

xuzheng0927 commented 7 years ago

My ARAP is working good now. I'll check (1) this evening.

On Mon, Nov 21, 2016 at 9:06 AM, Andrea notifications@github.com wrote:

  1. biharmonic deformation: you are using a SPD (symmetric positive definite) factorizer. Is your system matrix SPD? (potentially not) The slides (fairing) and the book's appendix cover how to correct this.
  2. ARAP: carefully read the end of Sec. 2 in the paper

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ataiya/dgp/issues/25#issuecomment-262000831, or mute the thread https://github.com/notifications/unsubscribe-auth/AI3gFNxtYZ13ZYGeWEd3X7oo7mCM9BQQks5rAc99gaJpZM4K30mw .

taiya commented 7 years ago

@xuzheng0927 before closing can you confirm what I was saying above addressed your problems?

xuzheng0927 commented 7 years ago

I can confirm that my problem is due to using a non-SPD matrix. But I haven't got the SPD implemented in my code (will do probably after the exam).

On Fri, Nov 25, 2016 at 9:30 AM, Andrea notifications@github.com wrote:

@xuzheng0927 https://github.com/xuzheng0927 before closing can you confirm what I was saying above addressed your problems?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ataiya/dgp/issues/25#issuecomment-263002188, or mute the thread https://github.com/notifications/unsubscribe-auth/AI3gFJCW8ksyoYrXb0KJsjTbNY2ijmyLks5rBxsqgaJpZM4K30mw .