(Towards #2381) Re-structure classes for reduction nodes.#3333
(Towards #2381) Re-structure classes for reduction nodes.#3333LonelyCat124 merged 37 commits intomasterfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3333 +/- ##
==========================================
- Coverage 99.95% 99.95% -0.01%
==========================================
Files 384 386 +2
Lines 54271 54187 -84
==========================================
- Hits 54249 54165 -84
Misses 22 22 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
It looks like the Met Office has removed one of the pages that the tutorial used to point to so I've updated the link. |
|
This time, the link-check failure is for a perfectly valid Issue on github so I think we can safely ignore it. |
LonelyCat124
left a comment
There was a problem hiding this comment.
Hi @arporter,
I'm not quite satisfied with the xfailing test - see inline.
|
Ready for another look/discussion about the test. |
|
Set CI running, otherwise I think everything is resolved. |
LonelyCat124
left a comment
There was a problem hiding this comment.
CI passed, ready to merge.
No description provided.