Conditional aux halo exchange via aux-field flag#55
Conversation
|
I'd prefer it if we could set the flag in the init routine of auxfield, that is we amend that one interface, but the others wouldn't need it anymore. This way the initialization of the type is not spread across multiple parts of the code (hopefully). |
|
@haraldkl the tracking is currently initialized after the auxfield, so |
|
Yes, though in that case it isn't that important, I still think it would be preferable to have function there setting the flag, rather than setting the flag directly. |
|
@haraldkl I’ve updated the implementation accordingly by introducing |
|
@MikeW097 I did some formatting. Please pull. Would you mind adding your copyright line in the top of the files where you added code? Otherwise that's fine to merge from my point of view. Thanks a lot! |
|
@haraldkl Thanks for the review and formatting fixes. I’ve added the copyright line to the files I modified. |
Based on the discussion in #53, the main goal of this update is to keep the existing interfaces unchanged to avoid unintended downstream effects in ApesMate while still enabling conditional auxiliary halo communication.