search

NUFgroup / IBPM.jl

Julia implementation of the Immersed Boundary Projection Method
3 stars 3 forks source link

operator-based analysis #9

Open ajgoza opened 3 years ago

ajgoza commented 3 years ago
jcallaham commented 2 years ago

Started working on this today. Following the framework above, I think the plan will be to use multiple dispatch within nonlinear to construct the linearized solver, then wrap it in a linear map. We'll also need to introduce a LinearizedIBProblem <: AbstractIBProblem that can store a base state (and ideally precompute the relevant arrays for the linearized advection term).

I haven't gotten as far as testing yet, just planning out how the functions will break down.

@ajgoza next time we talk we should take a look at the plan for the modified nonlinear function and make sure it's correct physically.

ajgoza commented 2 years ago

Yes, sounds good (to the plan and to the intention to discuss the nonlinear function in our meeting)!

On Sep 23, 2021, at 3:02 PM, Jared Callaham @.***> wrote:

Started working on this today. Following the framework above, I think the plan will be to use multiple dispatch within nonlinear to construct the linearized solver, then wrap it in a linear map. We'll also need to introduce a LinearizedIBProblem <: AbstractIBProblem that can store a base state (and ideally precompute the relevant arrays for the linearized advection term).

I haven't gotten as far as testing yet, just planning out how the functions will break down.

@ajgoza https://github.com/ajgoza next time we talk we should take a look at the plan for the modified nonlinear function and make sure it's correct physically.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NUFgroup/IBPM.jl/issues/9#issuecomment-926113576, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFREW53RQILBD5JWY3ERI5DUDOBURANCNFSM42JAWXSQ. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

jcallaham commented 2 years ago

Merged back into main. Almost good to go (at least for direct stability analysis), but I want to troubleshoot the asymmetry in the nonlinear term before we go any farther with it (see #32)