Skip to content

Pipeline Operation Normalisation tests#252

Open
edoyango wants to merge 4 commits intoACCESS-Community-Hub:developfrom
edoyango:norm-tests
Open

Pipeline Operation Normalisation tests#252
edoyango wants to merge 4 commits intoACCESS-Community-Hub:developfrom
edoyango:norm-tests

Conversation

@edoyango
Copy link
Copy Markdown
Collaborator

@edoyango edoyango commented Apr 7, 2026

This PR implements the tests for numpy, dask, xarray normalisation operations. a few # pragma: nocover were added for abstract method bodies and debug code.

Gemini 3.1 Pro was used to port the numpy tests to dask and xarray, since the operations were very similar.

Getting Started

  • If there is not an existing issue, raise a new issue
  • In the issue, state that you are intending to work on a contribution. This gives everyone involved the opportunity to discuss the best way forward.

Docstrings

  • Docstrings complete and follow Napoleon (google) style

Test Coverage

  • All new code is covered by unit tests

Documentation

  • Documentation is updated as required

Final Checks:

  • If no generative AI was used, then tick this box

Alternatively, if generative AI was used, then confirm you have:

  • Attributed any generative AI (such as GitHub Copilot) that was used in this PR. See our contributing guide for more information.
  • Included the name and version of the tool or system in the pull request
  • Described the scope of that use

Finally,

  • Mark the PR as ready to review. Note - we encourage you to ask for feedback at the outset or at any time during the work.

@edoyango edoyango force-pushed the norm-tests branch 2 times, most recently from 5452d4d to f9e0a19 Compare April 7, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant