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

Update the manual section on tests. #1811

Merged
merged 1 commit into from Jun 10, 2017
Merged

Conversation

bangerth
Copy link
Contributor

@bangerth bangerth commented Jun 8, 2017

Prompted by a recent mailing list discussion.

Copy link
Member

@tjhei tjhei 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 otherwise

@@ -1737,7 +1737,7 @@ \subsubsection{The Boussinesq approximation (BA)}
satisfies a relationship of the form $\rho=\rho(T)=\rho_0(1-\alpha(T-T_0))$
with a small thermal expansion coefficient $\alpha$ and a reference density
$\rho_0$ that is attained at temperature $T_0$. Since the thermal expansion is
considered small, this naturally leads to the following variant of the Boussinesq
considered small, this naturally leads to the following variant of the the Boussinesq
Copy link
Member

Choose a reason for hiding this comment

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

you seem to be undoing some useful prior changes here?!

@@ -6417,7 +6417,7 @@ \subsubsection{Tracking finite strain}
The plugin was tested against analytical solutions for the deformation gradient tensor in simple and pure shear as described in \url{benchmarks/finite_strain/pure_shear.prm} and \url{benchmarks/finite_strain/simple_shear.prm}.

We will demonstrate its use at the example of a 2D Cartesian convection model (Figure~\ref{fig:finite_strain}):
Heating from the bottom leads to the ascent of plumes from the boundary layer (top panel), and the
Heating from the bottom leads to the the ascent of plumes from the boundary layer (top panel), and the
Copy link
Member

Choose a reason for hiding this comment

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

same

@@ -9703,7 +9703,7 @@ \subsection{The idea of plugins and the \texttt{SimulatorAccess} and \texttt{Int

\begin{figure}[tbp]
\centering
\includegraphics[width=0.95\textwidth]{plugin_graph.pdf}
\includegraphics[width=0.95\textwidth]{plugin_graph}
Copy link
Member

Choose a reason for hiding this comment

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

same

\begin{verbatim}
\end{lstlisting}
in the build directory. To run the entire set of tests, then execute
\begin{lstlisting}[frame=single,language=ksh]
ctest
Copy link
Member

Choose a reason for hiding this comment

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

maybe also add "-j <number_of_cores>" here?

@bangerth
Copy link
Contributor Author

bangerth commented Jun 9, 2017

I must have accidentally saved an editor window after doing a pull. In any case, I've fixed the issues and added the -j description.

@tjhei tjhei merged commit 6817984 into geodynamics:master Jun 10, 2017
@bangerth bangerth deleted the doc-update branch June 11, 2017 16:19
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.

None yet

2 participants