<div dir="ltr"><div>Actually, that is nothing new at all. Its been that way "forever".<br><br>In the sequential case, one wants to use the default CollectiveCommunication (as defined in collectivecommunication.hh). To get that default implementation, a template parameter is to be given, that ensures that there is no specialization, that actually does something. Yasp uses its own name to do that. Any other name other than MPI_Comm would do.<br><br></div>I will have a look into vtktest to check what is wrong there.<br><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 3, 2014 at 11:23 AM, Markus Blatt <span dir="ltr"><<a href="mailto:markus@dr-blatt.de" target="_blank">markus@dr-blatt.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">New commit, appeared at Fri Oct 3 11:23:45 2014 +0200<br>
as part of the following ref changes:<br>
<br>
branch refs/heads/master updated from a3ff58a -> f6165ab<br>
<br>
Browsable version: <a href="http://cgit.dune-project.org/repositories/dune-grid/commit/?id=f6165aba501863dce958719ebbc2103ae8a646da" target="_blank">http://cgit.dune-project.org/repositories/dune-grid/commit/?id=f6165aba501863dce958719ebbc2103ae8a646da</a><br>
<br>
======================================================================<br>
<br>
commit f6165aba501863dce958719ebbc2103ae8a646da<br>
Author: Markus Blatt <<a href="mailto:markus@dr-blatt.de" target="_blank">markus@dr-blatt.de</a>><br>
Date: Fri Oct 3 11:23:02 2014 +0200<br>
<br>
[yaspgrid,bugfix] Use canonical communicator type for non-MPI case<br>
<br>
If people write programs the DUNE way then they will end up with<br>
Dune::FakeMPIHelper::MPICommunicator as their communicator type.<br>
Unfortunately Yaspgrid uses YaspGrid<dim, Coordinates> which is far<br>
from being inituitive and broke vtktest. This patch resorts to using<br>
the canonical choice, Dune::FakeMPIHelper::MPICommunicator, and fixes<br>
the test.<br>
<br>
dune/grid/yaspgrid.hh | 4 ++--<br>
1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
<br>
<br>
diff --git a/dune/grid/yaspgrid.hh b/dune/grid/yaspgrid.hh<br>
index f54fe6f..832ab7c 100644<br>
--- a/dune/grid/yaspgrid.hh<br>
+++ b/dune/grid/yaspgrid.hh<br>
@@ -89,7 +89,7 @@ namespace Dune {<br>
#if HAVE_MPI<br>
typedef CollectiveCommunication<MPI_Comm> CCType;<br>
#else<br>
- typedef CollectiveCommunication<Dune::YaspGrid<dim, Coordinates> > CCType;<br>
+ typedef CollectiveCommunication<FakeMPIHelper::MPICommunicator> CCType;<br>
#endif<br>
<br>
typedef GridTraits<dim, // dimension of the grid<br>
@@ -167,7 +167,7 @@ namespace Dune {<br>
#ifdef HAVE_MPI<br>
typedef CollectiveCommunication<MPI_Comm> CollectiveCommunicationType;<br>
#else<br>
- typedef CollectiveCommunication<YaspGrid<dim, Coordinates> > CollectiveCommunicationType;<br>
+ typedef CollectiveCommunication<FakeMPIHelper::MPICommunicator> CollectiveCommunicationType;<br>
#endif<br>
<br>
#ifndef DOXYGEN<br>
<br>
_______________________________________________<br>
Dune-Commit mailing list<br>
<a href="mailto:Dune-Commit@dune-project.org" target="_blank">Dune-Commit@dune-project.org</a><br>
<a href="http://lists.dune-project.org/mailman/listinfo/dune-commit" target="_blank">http://lists.dune-project.org/mailman/listinfo/dune-commit</a><br>
</blockquote></div><br></div></div>