Rob Falgout

Hi @fredrikekre . Thanks for pointing this out. It's inconsistent with what we do in our Krylov interfaces and StructHybrid as well. However, changing it would introduce memory leaks in...

There is a long discussion about this in Issue #513. The problem is the monolithic approach for compiling, which isn't scalable or maintainable. You should not point to any of...

@tzanio . Do you have anything to suggest here? Thanks!

Hi Florian, The `AMSProjectOutGradients` function is used to remove null-space components from the solution vector when the system is singular. It doesn't make the system nonsingular or anything. You can...

Hi Florian, I think the answer to your question is "no, boundary nodes are not internal nodes", because the matrix will have contributions from the nonzero beta region there. Let...

Looks like a typo to me. I would change it to `nVtemp2_l` as you suggest.

Hi @david8dixon and @jbschroder . After staring at this for a while, I guess I don't see a strong enough advantage to justify adding it to braid. Also, I think...

I would not use 'idx' for the coarse point. It looks as though it would make sense to provide both the fine and corresponding coarse indexes in CoarsenRefStatus. So, I...

Hi @jbschroder . This is better, but let me take a stab at it and get back to you.

Hi @jbschroder . Are you basically wanting the option to do `_braid_ComputeFullRNorm`? Maybe an alternative is to have the call to FAccess() be inside an if/else based on the option...