Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tutorial: partitioned oscillator #297

Conversation

BenjaminRodenberg
Copy link
Member

@BenjaminRodenberg BenjaminRodenberg commented Sep 30, 2022

* Tutorial illustrates a simple case which can be used for convergence
studies and waveforms
* Case is taken from V. Schüller, B. Rodenberg, B. Uekermann and H. Bungartz, A Simple Test Case for Convergence Order in Time and Energy Conservation of Black-Box Coupling Schemes, in: WCCM-APCOM2022. URL https://www.scipedia.com/public/Rodenberg_2022a
@BenjaminRodenberg
Copy link
Member Author

@valentinaschueller: Feel free to take a look and give comments, if you want 😁

mass-spring-1d/README.md Outdated Show resolved Hide resolved
@BenjaminRodenberg BenjaminRodenberg changed the title Add tutorial: partitioned mass spring system Add tutorial: partitioned oscillator Oct 3, 2022
@BenjaminRodenberg
Copy link
Member Author

  • I decided to rename this tutorial, because we also call it the Oscillator example in the paper. It's also a shorter name.
  • I also added the final result in 1c31544 to improve the plot on energy conservation.

Copy link
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.

Nice job, thanks for adding this. A few minor tweaks below.

Important problem. With preCICE v2, I run into:

(0) 14:51:57 [impl::SolverInterfaceImpl]:587 in isActionRequired: ERROR: initialize() has to be called before isActionRequired(...).

Seems that this tutorials only works with develop-v3.0.0.

I did not yet check all details.

oscillator/README.md Outdated Show resolved Hide resolved
oscillator/README.md Outdated Show resolved Hide resolved
oscillator/README.md Outdated Show resolved Hide resolved
oscillator/README.md Outdated Show resolved Hide resolved
oscillator/README.md Outdated Show resolved Hide resolved
oscillator/README.md Show resolved Hide resolved
oscillator/README.md Outdated Show resolved Hide resolved
oscillator/python/clean.sh Outdated Show resolved Hide resolved
oscillator/precice-config.xml Outdated Show resolved Hide resolved
oscillator/precice-config.xml Outdated Show resolved Hide resolved
Copy link
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.

Runs for me. A small final doubt below.

oscillator/python/oscillator.py Outdated Show resolved Hide resolved
oscillator/python/oscillator.py Outdated Show resolved Hide resolved
partitioned-heat-conduction/precice-config.xml Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants