[Dune] [#1011] operator* in densevector used as dot product in dune-istl is not suitable for complex vectors

Dune flyspray at dune-project.org
Mon Jan 9 12:16:55 CET 2012


THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

The following task has a new comment added:

FS#1011 - operator* in densevector used as dot product in dune-istl is not suitable for complex vectors
User who did this - Matthias Wohlmuth (matthias)

----------
Nothing's wrong about this fix. And probably it's the way to go,as it is the least intrusive solution. 

If the user is first reads through the DOKU, and is always aware of that, there is no problem at all I'm just not sure if this can be always assumed. So, if you're not, then for u complex, n real

int ( grad(u) * normal ) != int ( normal * grad(u)) . 

To implement this case, you would always have to make sure to conjugate the (real) normal, to not obtain an additional minus.  
I just think, this is hard to trace back as you won't get any obvious error messages. I had to debug that problem using a patched version as you proposed. Due to operator overloading, I was not aware, that at the end this operator was called in my expression. So, IMHO, it would be more robust to have to different operators similar to umhv and umtv for BCRS matrices. 



----------

More information can be found at the following URL:
http://www.dune-project.org/flyspray/index.php?do=details&task_id=1011#comment3019

You are receiving this message because you have requested it from the Flyspray bugtracking system.  If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.




More information about the Dune mailing list