-
-
Notifications
You must be signed in to change notification settings - Fork 64
v3: Make dt mandatory in documentation #257
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
Merged
Merged
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
0b2b4de
v3: Make dt mandatory in documentation
BenjaminRodenberg 1b159f6
Update read data functions to use relativeReadTime.
BenjaminRodenberg b2bf7b7
Merge branch 'master' into v3-make-dt-mandatory
BenjaminRodenberg 4027e53
Merge branch 'master' into v3-make-dt-mandatory
BenjaminRodenberg 78f580c
Merge branch 'master' into v3-make-dt-mandatory
BenjaminRodenberg 03746d0
Merge branch 'master' into v3-make-dt-mandatory
BenjaminRodenberg 700a13a
Add missing.
BenjaminRodenberg 33b1d8a
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg 9e74c64
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg f44389e
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg d280ec6
Shorten a bit.
BenjaminRodenberg a97b745
Minor follow-up for #258.
BenjaminRodenberg fe8db64
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg 680eee7
Add figure.
BenjaminRodenberg 6727496
Remove outdated note.
BenjaminRodenberg 0d44cfe
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg 3635024
Use dt properly.
BenjaminRodenberg 0809251
Remove unnecessary pdf.
BenjaminRodenberg 4d67208
Partial update of documentation w.r.t breaking changes.
BenjaminRodenberg 02cc542
Redice diff.
BenjaminRodenberg 2151663
Redice diff.
BenjaminRodenberg ecf9577
Reduce diff.
BenjaminRodenberg 78b625d
Fix some more breaking changes.
BenjaminRodenberg 9003fce
Add how dt is computed. See https://github.com/precice/precice.github…
BenjaminRodenberg f9e2f25
Merge branch 'update_breaking_changes' into v3-make-dt-mandatory
BenjaminRodenberg c5b6b0f
Apply suggestions from code review
BenjaminRodenberg 84e9caf
Merge branch 'update_breaking_changes' into v3-make-dt-mandatory
BenjaminRodenberg ab6a46f
Remove unneeded.
BenjaminRodenberg eea944b
Merge branch 'update_breaking_changes' into v3-make-dt-mandatory
BenjaminRodenberg a4950f6
Add pointer to interpolation section.
BenjaminRodenberg 34e7bea
Merge branch 'precice-v3' into update_breaking_changes
BenjaminRodenberg 83878b5
Merge branch 'update_breaking_changes' into v3-make-dt-mandatory
BenjaminRodenberg 64c796c
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg eb4f670
Fix format.
BenjaminRodenberg 6e08569
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg 35164be
Add advice on https://github.com/precice/precice/issues/1866.
BenjaminRodenberg 01f1b60
Apply suggestions from code review
BenjaminRodenberg d828aa9
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg 066e694
Update some minor, obviously outdated points.
BenjaminRodenberg 46aebdb
Update png.
BenjaminRodenberg 04101eb
Update pages/docs/couple-your-code/couple-your-code-mesh-and-data-acc…
BenjaminRodenberg 5cf756d
Update heading.
BenjaminRodenberg 5064011
Merge branch 'v3-make-dt-mandatory' of github.com:precice/precice.git…
BenjaminRodenberg 63b8145
Moved changes to #326.
BenjaminRodenberg 85ed50a
Merge branch 'precice-v3' into v3-make-dt-mandatory
BenjaminRodenberg 7dd94f2
Update pages/docs/couple-your-code/couple-your-code-mesh-and-data-acc…
uekerman e9abc5b
Update pages/docs/couple-your-code/couple-your-code-waveform.md
uekerman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Binary file added
BIN
+46.7 KB
images/docs/couple-your-code/couple-your-code-waveform/APIRelativeReadTime.pdf
Binary file not shown.
Binary file added
BIN
+22.7 KB
images/docs/couple-your-code/couple-your-code-waveform/APIRelativeReadTime.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
43 changes: 43 additions & 0 deletions
43
images/docs/couple-your-code/couple-your-code-waveform/APIRelativeReadTime.tex
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,43 @@ | ||
\documentclass[]{standalone} | ||
|
||
\usepackage{tikz} | ||
\usepackage{pgf} | ||
\usepackage{pgfplots} | ||
\usetikzlibrary{positioning, calc, decorations.markings} | ||
|
||
\begin{document} | ||
|
||
\begin{tikzpicture}[scale=10] | ||
\node[circle, fill=black, label={[xshift=-1em]below right:$\vphantom{\frac{1}{2}}t_{n+1} = t_{n} + \Delta t$}](tnp1) at (1.3,0) {}; | ||
\node[circle, fill=black, label={[xshift=-1em]below right:$\vphantom{\frac{1}{2}}\tau_{n}$}](taun) at (0,0) {}; | ||
\node[circle, fill=black, label={[xshift=-1em]below right:$\vphantom{\frac{1}{2}}\tau_{n+1} = \tau_n + \delta t$}](taunp1) at (0.35,0) {}; | ||
|
||
\draw[->](taun) edge[out=30,in=150] node[above, align=left, anchor=south west,xshift=-1cm]{(1) No more substeps: \\ \scriptsize{\texttt{precice.advance(preciceDt)}}}(tnp1); | ||
\draw[->](taun) edge[out=30,in=150] node[right, align=left, anchor=south west,pos=1]{(2) More substeps:\\ \scriptsize{\texttt{precice.advance(solverDt)}}} (taunp1); | ||
\node[above = 3cm of taun, label={[xshift=-1em, align=left]above right:Read interpolated data from current time $\tau_n$:\\ \scriptsize\texttt{precice.readData(relativeReadTime=dt, ...)}}](tau) {}; | ||
\draw[->](tau) -- (taun); | ||
|
||
\node[below = of taun, label={[xshift=-1em,yshift=0.1cm]below right:\scriptsize\texttt{dt = 0}}](rdt0) {}; | ||
\draw[->](rdt0) -- ($(taun)-(0,.2em)$); | ||
|
||
\draw[dotted] (taun) -- (taunp1); | ||
\draw[dotted] (taunp1) -- (tnp1); | ||
|
||
\node[circle, draw=black, fill=white, label={[xshift=-3.25em]below right:$\frac{1}{2}\left(\tau_n + \tau_{n+1}\right)$}](subcyclingmid) at ($0.5*(taun) + 0.5*(taunp1)$) {}; | ||
\node[below = 4.5em of subcyclingmid, label={[xshift=-1em,yshift=0.2cm]below right:{\scriptsize\texttt{dt = 0.5 * solverDt}}}](rdtmid) {}; | ||
\draw[->](rdtmid) -- ($(subcyclingmid)-(0,.2em)$); | ||
|
||
\node[below = of taunp1, label={[xshift=-1em,yshift=0.2cm]below right:\scriptsize\texttt{dt = solverDt}}](rdt1) {}; | ||
\draw[->](rdt1) -- ($(taunp1)-(0,.2em)$); | ||
|
||
\node[below = of tnp1, label={[align=center,xshift=-1em,yshift=0.2cm]below right:\scriptsize\texttt{dt = preciceDt}}](rdtend) {}; | ||
\draw[->](rdtend) -- ($(tnp1)-(0,.2em)$); | ||
|
||
|
||
\draw [decorate,decoration={brace,amplitude=2em,mirror}] ($(taun)-(0,.3)$) -- ($(taunp1)-(0,.3)$) node [black,midway,below,yshift=-2em, align=left] {Solver time step size $\delta t$\\ \scriptsize\texttt{solverDt < preciceDt} }; | ||
|
||
\draw [decorate,decoration={brace,amplitude=2em,mirror}] ($(taun)-(0,.25)$) -- ($(tnp1)-(0,.25)$) node [black,midway,below,yshift=-2em, xshift=5em, align=left] {Complete time window size $\Delta t$ or remainder\\ \scriptsize\texttt{preciceDt = precice.getMaxTimeStepSize()}}; | ||
\end{tikzpicture} | ||
|
||
\end{document} | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think that this is a good strategy (see also above): To tell users first how to do it the wrong way, and then the right way. Maybe we need to re-order our steps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that the waveform API is not experimental anymore but the default, let's put some information from here into "Step 5 - Non-matching time step sizes". I think this section needs some updates and refinement anyway. Additionally, I think that the topics that we currently explain in "Step 5 - Non-matching time step sizes" are definitely more advanced and exotic than some of the basic topics that we explain here. So yes: let's reorder some things.
We should still keep the more advanced background information (interpolation degree, higher-order time stepping) here to not provide too much information too early.