Skip to content

DELWAQ-1234: Update the source code and introduce auxiliary routines#999

Open
arjenmarkusdeltaresnl wants to merge 8 commits into
mainfrom
waq/bugfix/DELWAQ-1234-install-auxiliary-routines-openpl
Open

DELWAQ-1234: Update the source code and introduce auxiliary routines#999
arjenmarkusdeltaresnl wants to merge 8 commits into
mainfrom
waq/bugfix/DELWAQ-1234-install-auxiliary-routines-openpl

Conversation

@arjenmarkusdeltaresnl

Copy link
Copy Markdown
Contributor

To demonstrate the use of the auxiliary routines I edited the original example: Write a message to the screen via the logger
Check if the segment is active
I had to add a surprising number of modules to get the precision working.

What was done

  • e.g. Restarts are made more robust
  • e.g. Fixes a bug related to the writing of water depth on the map file
  • e.g. Introduces a new functionality on energy losses at bridge piers
  • e.g. …

Evidence of the work done

  • Video/figures
    <add video/figures if applicable>
  • Clear from the issue description
  • Not applicable

Tests

  • Tests updated
    <add testcase numbers if applicable, Issue number>
  • Not applicable

Documentation

  • Documentation updated
    <add description of changes if applicable, Issue number>
  • Not applicable

Issue link

To demonstrate the use of the auxiliary routines I edited the original example:
Write a message to the screen via the logger
Check if the segment is active
I had to add a surprising number of modules to get the precision working.
Curiously enough, the example was never installed. This change makes good on that.
To avoid confusion only include the source files that are really needed.

@michelle-deltares michelle-deltares left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants