aspect icon indicating copy to clipboard operation
aspect copied to clipboard

Published 20 hours ago • verified publisher icon geodynamics

replace ParticleOutput by deal.II::Particle::DataOut

Open tjhei opened this issue 2 years ago • 3 comments

Is there anything missing in the deal.II version of this class? It was already available in 9.3...

https://github.com/geodynamics/aspect/blob/c9601fad8357f73b6c19bc396d6ef211a1585b03/include/aspect/postprocess/particles.h#L50

@gassmoeller ?

tjhei avatar Feb 10 '23 12:02 tjhei

Yes, the ASPECT version has the option to exclude certain particle properties from the output (by name). We have use cases with dozens of particle properties where we want to output only certain properties. I am not sure if that is something that would be useful for deal.II too. Maybe there are other (small) differences too. It would be useful to unify them though.

gassmoeller avatar Feb 10 '23 13:02 gassmoeller

@gassmoeller Want to keep this open?

Is outputting all particle properties an issue with disk space? Or why do you want to keep this option?

bangerth avatar Jul 13 '23 23:07 bangerth

Yes I think the disk space is an issue, especially for the models of @MFraters. Let's keep this open, I would still like to unify the ASPECT and deal.II classes.

gassmoeller avatar Jul 15 '23 01:07 gassmoeller