Skip to content

Conversation

@mtar
Copy link
Collaborator

@mtar mtar commented Nov 28, 2025

Due Diligence

  • General:
  • Implementation:
    • unit tests: all split configurations tested
    • unit tests: multiple dtypes tested
    • NEW unit tests: MPS tested (1 MPI process, 1 GPU)
    • benchmarks: created for new functionality
    • benchmarks: performance improved or maintained
    • documentation updated where needed

Description

Issue/s resolved: #1924

Changes proposed:

Type of change

Memory requirements

Performance

Does this change modify the behaviour of other functions? If so, which?

yes / no

@github-actions github-actions bot added the utils label Nov 28, 2025
@mtar mtar changed the title move tests move tests to separate directory Dec 1, 2025
@mtar mtar marked this pull request as ready for review December 1, 2025 08:54
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

Thank you for the PR!

@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

Thank you for the PR!

@github-actions
Copy link
Contributor

github-actions bot commented Dec 3, 2025

Thank you for the PR!

@github-actions
Copy link
Contributor

Thank you for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

Separate tests from code

2 participants