(Closes #868) implement support for kernels with field arguments that specify nlayers/ndata - #3510
(Closes #868) implement support for kernels with field arguments that specify nlayers/ndata#3510arporter wants to merge 32 commits into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3510 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 399 399
Lines 56094 56196 +102
==========================================
+ Hits 56094 56196 +102 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…eight_named_values to cover nlayers/ndata; lib/process.py: guard CLI parsing under __main__ Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
This is very close to being ready. I need to finish doing a 'self review' and deal with the various TODOs, esp. where I'm using |
|
This is ready for a first review now. It's slightly bigger (in terms of files touched) than I would like because I realised that at some point during the last PR, I mixed 'nlevels' with 'nlayers'. I've gone through and fixed this so that we only use 'nlayers' now. I've also altered the naming used for all |
|
ITs were all green. @stevemullerworth and @christophermaynard it would be good if someone at the MO could also test this branch out to check that I haven't missed anything. |
No description provided.