<div dir="ltr"><div>Hey Oliver,<br></div><div>this is indeed something to think about.<br></div><div>I have bad news though. I have done a quick measurement of option b).<br></div><div>I added a parameter origin to EquidistantCoordinates and defaulted it to 0.<br></div><div>This slowed down the finitevolume example by roughly 10%. </div><div>You're option a) is of course VERY simple to implement. The only question<br>that bugs me about it, is how to pass the additional information. I would be very happy <br></div><div>if there was a good way, that does not involve introducing an additional constructor.<br></div><div>Any opinions?<br></div>Best,<br>Dominic</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 10, 2014 at 7:13 PM, Oliver Sander <span dir="ltr"><<a href="mailto:sander@igpm.rwth-aachen.de" target="_blank">sander@igpm.rwth-aachen.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
while thinking about how to replace SGrid by YaspGrid I came about the following<br>
small inconvenience: SGrid are uniformly spaced, and both the lower left and the<br>
upper right corners are free.  YaspGrid, OTOH, when they are not tensor-product grids,<br>
constrain the lower-left corner to be (0,...,0).  So currently, in order to replace<br>
SGrids I sometimes have to use tensor-product YaspGrids even though they are uniformly<br>
spaced.<br>
Should we<br>
a) introduce a third Coordinates class in YaspGrid for uniformly spaced grids with<br>
   a nonzero left corner?<br>
b) generalize the current EquidistantCoordinate class to allow for a nonzero left<br>
   corner?<br>
<br>
a) would be preferable if b) would lead to a measurable slowdown.  Does anybody<br>
have experience with this?<br>
Thanks,<br>
Oliver<br>
<br>
<br>_______________________________________________<br>
Dune-devel mailing list<br>
<a href="mailto:Dune-devel@dune-project.org">Dune-devel@dune-project.org</a><br>
<a href="http://lists.dune-project.org/mailman/listinfo/dune-devel" target="_blank">http://lists.dune-project.org/mailman/listinfo/dune-devel</a><br>
<br></blockquote></div><br></div>