Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

Fix TestGaussianGateApplication tests - #762

Closed
heltluke wants to merge 10 commits into
XanaduAI:masterfrom
CatalinaAlbornoz:fix-testgaussiangateapplication-tests
Closed

Fix TestGaussianGateApplication tests#762
heltluke wants to merge 10 commits into
XanaduAI:masterfrom
CatalinaAlbornoz:fix-testgaussiangateapplication-tests

Conversation

@heltluke

Copy link
Copy Markdown
Contributor

Context:
Two tests are failing with ValueError: The matrix S is not symplectic.

Description of the Change:
Reduces the scale from 1.0 to 0.5 in the failing tests, i.e. S = tf.Variable(random_symplectic(num_mode, scale=0.5), dtype=tf.complex128)

Benefits:
Tests pass

Possible Drawbacks:
We're not testing what we should be testing?

Related GitHub Issues:
N/A

CatalinaAlbornoz and others added 10 commits November 27, 2025 13:35
Make it clear that we're not accepting contributions for new features.
Clarify that we're not accepting contributions for new features.
Make it clear that we're not accepting new features
make it clear that we're not accepting new features
@heltluke heltluke changed the title Fix testgaussiangateapplication tests Fix TestGaussianGateApplication tests Dec 19, 2025
@heltluke
heltluke marked this pull request as draft December 19, 2025 15:44
@codecov

codecov Bot commented Dec 19, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.21%. Comparing base (9a9a352) to head (0449e0d).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #762      +/-   ##
==========================================
- Coverage   98.33%   98.21%   -0.12%     
==========================================
  Files          78       78              
  Lines        9603     9539      -64     
==========================================
- Hits         9443     9369      -74     
- Misses        160      170      +10     

see 74 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc0a0e4...0449e0d. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@heltluke heltluke closed this Dec 19, 2025
@heltluke
heltluke deleted the fix-testgaussiangateapplication-tests branch December 19, 2025 15:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants