fischer1

Invalid Jacobian = Abort flag

2

Nek checks the mesh Jacobian for negative or 0 values at several stages. For most scenarios, only one such check is worthy of aborting the run, that is just prior...

L1/L2 diagnostics do not show timestep number

1

Can we modify the Nek5000 source so that L1/L2 DIV(V)-QTL -1.2948E-20 2.2554E-05 L1/L2 DIV(V) -1.0147E-19 2.2191E-05 L1/L2 QTL 0.0000E+00 0.0000E+00 L1/L2 DIV(V)-QTL -1.0147E-19 2.2191E-05 will be preceeded by timestep number?...

rdparam references nelgv before it is read

2

These are just diagnostics, so don't really impact code function... but are bugs in any case: if (ifneknekc.and.(nelgv.ne.nelgt)) call exitti( $ 'ABORT: nek-nek not supported w/ conj. ht transfer$',1) if...

conjugate heat transfer for genbox

1

I have a version of conjugate heat transfer for genbox. You simply put an "s" prefix (for "solid") in front of the standard box or circle statement. All elements generated...

genmap + periodic is broken for thermal only cases (i.e., ifflow=F)

We should probably check when ifheat=T to make sure that p8 .ne. 0 Otherwise, it messes up the velocity field... :(

NekRS currently writes the total flops for the simulation (presumably skipping the setup phase). It would be nice to also have "flops per rank" written out so that grep -iIR...

NekExamples/benard/ray_9 fails

1

This case runs with v17, but blows up in ~9 steps with the current repo version. Can we test all examples that are in NekExamples, even if infrequently? These days...

IFCHAR = T ??

3

I notice that several examples have ifchar=T. Some don't even use a dt big enough to warrant such usage (e.g., cyl_restart case). I think all examples (except for short tests...