Peter Hill

Results 535 comments of Peter Hill

Excellent, pleased to hear it! I hadn't realised they were official -- probably doesn't make sense to include them in the corpus tests just yet then

Suggestions for deletion: - 2Dturbulence_multigrid - Demonstrates a solver, could done with an input file for a different model - Move to separate repo? - advdiff - I don't think...

This looks massive, but I guess it's because it includes `next` or something into the base branch? @dschwoerer Please could you update the base branch so the changes here are...

There's some work currently in place to refactor `Coordinates` that will hopefully eliminate the need for `geometry()` entirely, so I think that's still the case.

No, the text is correct for 5.0.0 (the section heading aside). The calls to `geometry()` have been removed from the `Coordinates` ctors, so it must be called manually -- but...

Looking at what [versions of SUNDIALS are available across OSes](https://pkgs.org/search/?q=sundials), it seems most have >= 5.0, so we could use that as our minimum version? There's still a couple of...

Yep, very much agree! I'd love to push some of this upstream, but their error system can only return a `const char*`, so they have no opportunity to customise the...

Hi @davedavemckay, sorry for the slow response! You're actually specifying the mesh in the input file: ```ini [mesh] nx = 260 # Note 4 guard cells in X ny =...

Good memory! Yes, although it looks like he had to use raw pointers in order to use `cudaMallocManaged` to allocate memory on the GPU. I guess we don't have that...