Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

code re-structuring #7

Open
4 tasks
ajgoza opened this issue Apr 2, 2021 · 2 comments
Open
4 tasks

code re-structuring #7

ajgoza opened this issue Apr 2, 2021 · 2 comments

Comments

@ajgoza
Copy link
Collaborator

ajgoza commented Apr 2, 2021

  • better names for types and structs?
  • maybe break up into several files based on the category of each type (so maybe grid types and structs in a file, motion types and structs in another, perhaps an entire separate one for cylinder types and motions?)
  • Combine IBmodel and IBproblem?
  • Possible renaming: AbstractIBProblem --> AbstractProblem, IBProblem --> Problem, State --> AbstractState --> IBState --> State
@ajgoza ajgoza changed the title problem-types.jl is getting clunky... clean up problem-types.jl... Apr 2, 2021
@jcallaham
Copy link
Collaborator

For IBModel vs IBProblem - I set it up this way as kind of a half-baked compromise between the "NavierStokesModel" in the C++ code and the "Problem" interface for DifferentialEquations.jl. Basically the idea was to try to contain the physical setup (compressibility/viscous, etc) in the "model" and do the time evolution with "problem". But now that I understand the methods a bit better I think the treatment of different physics might be too different to be conveniently abstracted like this. Also so far nobody has mentioned interest in doing anything but incompressible viscous flow, so maybe it's not worth worrying about too much.

In other words, at the moment there's no clear conceptual distinction to me between what should be at the "model" vs "problem" level, but I can't tell whether or not that would change with likely future work. Any thoughts?

@ajgoza
Copy link
Collaborator Author

ajgoza commented Apr 5, 2021

I see the distinction you're drawing now. I agree with your assessments that (i) the distinction is not especially necessary now, and (ii) it might be useful depending on changes we make moving forward. Let's chat more tomorrow

@ajgoza ajgoza changed the title clean up problem-types.jl... code re-structuring Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants