Skip to content

Make read mapping conditional.#1404

Merged
BenjaminRodenberg merged 12 commits intoprecice:develop-v3.0.0from
BenjaminRodenberg:834-check-unconditional-read-mapping-in-initializedata
Aug 8, 2022
Merged

Make read mapping conditional.#1404
BenjaminRodenberg merged 12 commits intoprecice:develop-v3.0.0from
BenjaminRodenberg:834-check-unconditional-read-mapping-in-initializedata

Conversation

@BenjaminRodenberg
Copy link
Copy Markdown
Contributor

@BenjaminRodenberg BenjaminRodenberg commented Aug 5, 2022

Main changes of this PR

Closes #834.

Expose CouplingScheme::sendsInitializedData to the public.
Check for sendsInitializedData to determine whether mapping is necessary.

Motivation and additional information

See #834.

Author's checklist

  • I added a changelog file with make changelog if there are user-observable changes since the last release. Not visible for users.
  • I added a test to cover the proposed changes in our test suite. I updated an existing test.
  • I ran make format to ensure everything is formatted correctly.
  • I sticked to C++14 features.
  • I sticked to CMake version 3.16.3.
  • I squashed / am about to squash all commits that should be seen as one.

Reviewers' checklist

  • Does the changelog entry make sense? Is it formatted correctly?
  • Do you understand the code changes?

BenjaminRodenberg and others added 7 commits August 5, 2022 12:20
* Remove mapReadDataTo and mapWriteDataFrom.
* Remove tests using removed API.
* Merge initialize and initializeData in SolverInterfaceImpl.
* Merge initialize and initializeData in CouplingScheme.
* Update tests correspondingly
* Allow to check actions before initialize, especially checking whether initial data has to be written
* Check for sending and receiving of initial data during coupling scheme configuration
* Code simplification w.r.t waveform.
* Use dt=0.0 in actions in initialize.

Co-authored-by: Benjamin Uekermann <[email protected]>
Co-authored-by: Frédéric Simonis <[email protected]>
…recice#1352)

* Get tests and configs for serial implicit waveform interpolation from precice#1029.
* Differentiate between received initial data and data (from advance).
* Split data communication for SerialCouplingScheme and store initial data in waveform.
* Move common functionality of serial and parallel coupling scheme into BiCouplingScheme.

Co-authored-by: Benjamin Uekermann <[email protected]>
* Add test Integration/Serial/InitializeData/ImplicitBoth where both participants initialize.
* Allow initialize=true for both participants in serial-implicit coupling scheme.
* Raise warning, if first participant in serial coupling scheme initializes data with waveform order zero.
* Remove check for write data, because participant does not know about waveform order.

Co-authored-by: Benjamin Uekermann <[email protected]>
@BenjaminRodenberg BenjaminRodenberg added the maintainability Working on this will make our lives easier in the long run as preCICE gets easier to maintain. label Aug 5, 2022
@BenjaminRodenberg BenjaminRodenberg self-assigned this Aug 5, 2022
@BenjaminRodenberg BenjaminRodenberg changed the base branch from develop to develop-v3.0.0 August 5, 2022 11:17
@BenjaminRodenberg BenjaminRodenberg added this to the Version 3.0.0 milestone Aug 5, 2022
Copy link
Copy Markdown
Member

@uekerman uekerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.
Probably worth a changelog entry.

@BenjaminRodenberg BenjaminRodenberg merged commit 72a6b20 into precice:develop-v3.0.0 Aug 8, 2022
BenjaminRodenberg added a commit that referenced this pull request Aug 11, 2022
* Make read mapping conditional to avoid unnecessary mappings.
* Update corresponding tests.

Co-authored-by: Benjamin Uekermann <[email protected]>
BenjaminRodenberg added a commit that referenced this pull request Aug 22, 2022
* Make read mapping conditional to avoid unnecessary mappings.
* Update corresponding tests.

Co-authored-by: Benjamin Uekermann <[email protected]>
BenjaminRodenberg added a commit that referenced this pull request Aug 22, 2022
* Make read mapping conditional to avoid unnecessary mappings.
* Update corresponding tests.

Co-authored-by: Benjamin Uekermann <[email protected]>
BenjaminRodenberg added a commit that referenced this pull request Sep 15, 2022
* Make read mapping conditional to avoid unnecessary mappings.
* Update corresponding tests.

Co-authored-by: Benjamin Uekermann <[email protected]>
BenjaminRodenberg added a commit that referenced this pull request Nov 2, 2022
* Make read mapping conditional to avoid unnecessary mappings.
* Update corresponding tests.

Co-authored-by: Benjamin Uekermann <[email protected]>
BenjaminRodenberg added a commit that referenced this pull request Nov 4, 2022
* Make read mapping conditional to avoid unnecessary mappings.
* Update corresponding tests.

Co-authored-by: Benjamin Uekermann <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainability Working on this will make our lives easier in the long run as preCICE gets easier to maintain.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check unconditional read mapping in initializeData

2 participants