Benjamin Uekermann
Benjamin Uekermann
> What about the distributed meshes? As far as I can see, if the scaling is calculated locally, I am not sure that the integral is conserved for the total...
Let's keep this open till we have the parallel version as well.
I guess if we do this change we could simply remove `hasReadMapping()` and `hasWriteMapping()`. These functions seem to have no real purpose anymore.
Are we sure we want this? See discussion in #1156 Let's try to first conclude there.
Let's conclude in #1156 first.
Not breaking in my opinion if we don't restrict real use cases.
https://github.com/precice/precice/blob/e00e332f67269db474b94858af76223be89e7b90/src/action/Action.hpp#L58-L62 I am not aware of any (real) use case of `timeStepSize` or `computedTimeWindowPart` or `timeWindowSize` here (but `time` yes). In principle, one could use this to compute velocity values...
> This means that the new signature will be preformAction(double time) and that's it, right? yes > We will also delete some actions then, but I'm fine with this. yes...
Sth to think about: when we make it mandatory, couldn't we fully integrate the functionality into `initialize`?