Skip to content

Conversation

@mcjamieson
Copy link

Summary

Added comprehensive input validation tests for quantum measurement and time evolution operations.

Changes

operations.cpp

  • Added 12 validation test sections covering measurement, projection, and PauliStrSum operations
  • Tests verify proper error handling for invalid quantum registers, out-of-bounds qubits, duplicate targets, and zero targets

trotterisation.cpp

  • Added complete test suite for unitary and imaginary time evolution operations
  • Includes 8 validation sections testing invalid inputs (bad registers, non-Hermitian Hamiltonians, invalid Trotter parameters)
  • Added 4 correctness test scenarios for imaginary time evolution: strong magnetic field, opposite magnetic field, ferromagnetic system, and antiferromagnetic system. Each verifies the algorithm converges to the expected lowest-energy configuration.

Test Results

  • All 12 operations validation tests pass
  • applyTrotterizedUnitaryTimeEvolution: 19 assertions
  • applyTrotterizedImaginaryTimeEvolution: 262,153 assertions

TysonRayJones and others added 4 commits May 28, 2025 03:06
Updates version references and fixes doxygen parsing of CMake configured header broken in QuEST-Kit#616
@mcjamieson mcjamieson marked this pull request as draft January 14, 2026 16:47
@mcjamieson mcjamieson marked this pull request as ready for review January 14, 2026 16:47
@otbrown otbrown self-assigned this Jan 14, 2026
@otbrown
Copy link
Collaborator

otbrown commented Jan 14, 2026

Thanks Maurice! I'll add reviewing this to my list.

@otbrown otbrown changed the base branch from main to devel January 14, 2026 17:33
@tom-quantummotion
Copy link

Thanks @mcjamieson!

@otbrown
Copy link
Collaborator

otbrown commented Jan 15, 2026

There is a PR to the PR which fixes conflicts arising from Vasco's concurrent changes to the time evolution API!

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.

4 participants