grid icon indicating copy to clipboard operation
grid copied to clipboard

Poisson Solver initial value problem implementation is not robust.

Open PaulWAyers opened this issue 2 years ago • 4 comments

The current boundary-value-problem for the Poisson solver is pretty robust. The initial value problem is a lot less robust.

It's a good first issue to try to make this work more efficiently. I will post some notes on how to do this.

PaulWAyers avatar May 11 '23 15:05 PaulWAyers

@PaulWAyers Can you please give me more details on it? I would like to work on it.

Aditish51 avatar Mar 15 '24 18:03 Aditish51

The current code uses a large initial interval (r_interval[0]) for the IVP solver. This might be unnecessary, especially if f decays rapidly with increasing r. Reducing the initial interval to a value just larger than the region of interest can potentially improve performance.

Charlie-1-3 avatar Mar 22 '24 21:03 Charlie-1-3

@Charlie-1-3 did you test it to see if this helped?

@Ali-Tehrani do you have some examples for this? I seem to recall that Carlos had an example that was problematic, but I don't remember for certain.

PaulWAyers avatar Mar 23 '24 13:03 PaulWAyers

@PaulWAyers Yes it worked for a few test cases where function f was inversely proportional to r. Cases, when f depends directly on r, might add extra computation in IVP because of larger domain traversal. Use of Relaxation Techniques for Poisson’s Equation one of which is Successive Over-Relaxation (SOR) would help too.

Charlie-1-3 avatar Mar 23 '24 21:03 Charlie-1-3