Closed lroberts36 closed 3 weeks ago
w.r.t. swarms, I tested this downstream and all looks good. I noticed that this PR is targeting lroberts36/add-fine-variables
instead of develop
... intentional?
I noticed that this PR is targeting
lroberts36/add-fine-variables
instead ofdevelop
... intentional?
@pdmullen: yeah, but only because I was using the new test I added there to check that removing the MeshBlockData
boiler plate worked.
@lroberts36 are we waiting for someone to review this?
@lroberts36 are we waiting for someone to review this?
@Yurlungur: @pgrete said he would on Thursday, but I think he has been under the weather the last few days.
"Remove Initialize methods and replace them with constructors since they break RAII" is possibly meant for a future PR, isn't it? In that case -> new issue?
@pgrete: I think I was just annoyed when I wrote this comment. It is probably not worth the effort to remove the initialization methods and replace them with constructors.
PR Summary
This PR makes some changes to
DataCollection
so that callingpmesh->mesh_data.GetOrAdd(some_label, partition)
works whether or notsome_label
has been previously added to theMeshBlockData
container. Additionally, it makes it so thatAdd
will now work correctly forMeshData
(previously it didn't use the correct labeling and containers could be overwritten).Possible other things to do:
Mesh
DataCollection
interface to work with bothMesh
andMeshData
Initialize
methods and replace them with constructors since they break RAIIPR Checklist