search

geodynamics / aspect

A parallel, extensible finite element code to simulate convection in both 2D and 3D models.
https://aspect.geodynamics.org/
Other
223 stars 235 forks source link

remove parameter-gui? #5706

Open tjhei opened 4 months ago

tjhei commented 4 months ago

I vote to remove cmake support for the QT based parameter GUI https://github.com/dealii/parameter_gui. Or is anyone using it?

bangerth commented 4 months ago

I thought I saw someone use it recently (on a Zoom call?). Isn't this a separate program now?

bangerth commented 4 months ago

In other words, what would we have to remove?

tjhei commented 4 months ago

https://github.com/geodynamics/aspect/blob/70e161d7b5ea5daf13680271f42e2ddd65c53d5d/CMakeLists.txt#L815-L844

bangerth commented 4 months ago

I don't think it's an issue that bothers me too much. Take a poll tomorrow morning!

tjhei commented 4 months ago

poll results: nobody uses the parameter gui.

gassmoeller commented 4 months ago

see my comment in #5735. What you are proposing is removing the configuration of ASPECT-GUI as part of ASPECT build. ASPECT-GUI works just fine separately.