adjtomo/seisflows

consolidate solver classes?

rmodrak opened this issue · 2 comments

Currently, there are three main solver classes: specfem2d, specfem3d, and specfem3d_globe. Would it make sense to have all three inherit from a single base class?

mpbl commented

I guess it suits our needs, at least for now.
Otherwise, you are probably right. Especially if you intend to target non specfem* solvers on the long term.

implemented in branch "refactor-solver"