Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Forwarding to v1.0.0 #37

Merged
merged 307 commits into from
Aug 15, 2024
Merged

Forwarding to v1.0.0 #37

merged 307 commits into from
Aug 15, 2024

Conversation

taha1373
Copy link
Collaborator

No description provided.

taha1373 and others added 30 commits October 28, 2022 17:00
@taha1373
clean ups
525840b
@taha1373
gitignore system_debuging.py
80248a7
@Alekxos
Change dec_rate variable operations from in-place to out-of-place
4fe1cb6
@Alekxos
Bring code up to PEP8 standard and make improvements to style
2e15fa0
@Alekxos
Minor update to T1 test for transmon, to include other types of T1 noise
572a61e
@Alekxos
Alter _update_H method to also include logic of set_trunc_nums within…
5a2beae 
… public method circuit.update(), and streamline tests accordingly
@Alekxos
Add support for handling negative unit values
0761345
@Alekxos
Update gradient tests with additional comments
ae9e020
@Alekxos
Add copy function to Circuit class; use this to introduce differentia…
ccdbc06 
…ble external flux sensitivity function and test its gradient
@Alekxos
Add support for numerical test of anharmonicity gradient
00fa824
@taha1373
Merge branch 'dev-tr' into dev-tq
5a267d6
@taha1373
Merge pull request #11 from stanfordLINQS/dev-tq
f9312be 
Backend and tests for autodifferentiation of T1 noise, combined eigensolver
@taha1373
added the test for system of three coupled Transmons
5ac9a9a
@Alekxos
Automatically enforce that element values lie above baseline, both wh…
600c61c 
…en updated after gradient step (via circuit.update()) and manually, when set via set_value; assign default values for each element type
@Alekxos
Merge code with systems update from dev-tr
6d6c982
@Alekxos
Fix error with passing empty list of tensors to eigencircuit solver
90fc52b
@Alekxos
Add additional warning when optimization leads to element value below…
2dee7b8 
… baseline
@Alekxos
Add minor fixes to negative value warnings
b6e8d80
@Alekxos
Fix minor minor issue with negative values
424607a
@Alekxos
Support random sampling
114130b
@Alekxos
Fix minor syntax issue
5ac6868
@Alekxos
Fix division by zero error
80f33f9
@Alekxos
Fix division by zero error (again)
3662bad
@Alekxos
Temporary debugging
e02c1ad
@Alekxos
Temporary debugging (x2)
82acf87
@Alekxos
Order sampled circuit elements
e738e82
@Alekxos
Debug optim mismatch
897a230
@Alekxos
Fix some issues with np vs. torch shapes, ensure topologies sorted co…
9f0ccf6 
…nsistently
@Alekxos
Fix one additional place where np/torch shapes were off by 1
c852ed7
@Alekxos
Add support for maximum element value
5edec44
taha1373 and others added 25 commits August 7, 2024 19:35
@taha1373
Merge pull request #36 from stanfordLINQS/dev-tq_Qfix
c291be1 
Merge in QuTiP 5.0.x compatibility updates
@sambonkov
Update pytest.yml
d5983a5
@sambonkov
Clean up calculations for description.
f76bfc1
@sambonkov
Test descriptions passes with reordering.
83604af
@sambonkov
with prior
8ba8eb4
@sambonkov
Hardcode linewidth for description with text.
6a45cdb
@taha1373
PEP8 and typo clean ups
e07409f
@taha1373
centering the logo in readme
ba24f24
@sambonkov
Update for SymPy 1.13 compatibility.
4be4b9f
@sambonkov
Typo fix; make sure all tensors are doubles
615c3ba
@sambonkov
Remove extra gap in description.
a37698d
@sambonkov
Add functionality to print element values.
4818dd2
@taha1373
PEP8 clean up
12959e2
@taha1373
Merge branch 'dev-tq' of https://github.com/stanfordLINQS/SQcircuit i…
ac68a9d 
…nto dev-tq
@taha1373
change readme
7a530e7
@taha1373
Forward v1.0.0
dc018a4
@taha1373
PEP8 clean ups
6c66f72
@taha1373
PEP8 clean up
d049943
@taha1373
PEP8 clean ups
3556d1d
@taha1373
fix a typo
977b3b3
@taha1373
PEP8 clean up
1358986
@taha1373
adding support for removing uncoupled mode back to numpy engine of SQ…
8503180 
…circuit
@taha1373
remove some printing related to debugging
11746f5
@taha1373
fixing the minus issue in junction part of the Hamiltonian in circuit…
15aa447 
… description.
@taha1373
add a note to readme file to say SQcircuit is compatible with QuTip v…
3a2a5f6 
…ersions 5.0 and above.
taha1373
taha1373 commented Aug 15, 2024
View reviewed changes
Copy link
Collaborator Author

@taha1373 taha1373 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good

@codecov Codecov
Copy link

codecov bot commented Aug 15, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 93.19923% with 142 lines in your changes missing coverage. Please review.

Project coverage is 89.58%. Comparing base (3f0e736) to head (a5cb5de).
Report is 308 commits behind head on main.

Files Patch % Lines
SQcircuit/functions.py 78.86% 52 Missing ⚠️
SQcircuit/tests/test_papers.py 67.81% 28 Missing ⚠️
SQcircuit/sweep.py 14.28% 12 Missing ⚠️
SQcircuit/elements.py 96.01% 8 Missing ⚠️
SQcircuit/torch_extensions.py 96.52% 8 Missing ⚠️
SQcircuit/storage.py 0.00% 7 Missing ⚠️
SQcircuit/tests/test_grad.py 97.48% 7 Missing ⚠️
SQcircuit/units.py 54.54% 5 Missing ⚠️
SQcircuit/exceptions.py 76.47% 4 Missing ⚠️
SQcircuit/symbolic.py 96.11% 4 Missing ⚠️
... and 5 more
Additional details and impacted files 
@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
+ Coverage   89.22%   89.58%   +0.36%     
==========================================
  Files          16       27      +11     
  Lines        1578     3467    +1889     
==========================================
+ Hits         1408     3106    +1698     
- Misses        170      361     +191
Flag Coverage Δ
unittests 89.58% <93.19%> (+0.36%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@taha1373 taha1373 merged commit 7cdc826 into main Aug 15, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@taha1373 @Alekxos @sambonkov