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.

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

Zqs 920 loss function wrapper #33

Merged
merged 8 commits into from
Jan 21, 2022
Merged

Conversation

mstechly
Copy link
Contributor

No description provided.

@mstechly mstechly requested a review from MSRudolph January 19, 2022 20:26
@codecov
Copy link

codecov bot commented Jan 19, 2022

Codecov Report

Merging #33 (c0cd140) into dev (473b8c6) will increase coverage by 0.68%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev      #33      +/-   ##
==========================================
+ Coverage   82.84%   83.52%   +0.68%     
==========================================
  Files          27       28       +1     
  Lines         653      680      +27     
  Branches       68       71       +3     
==========================================
+ Hits          541      568      +27     
  Misses         92       92              
  Partials       20       20              
Impacted Files Coverage Δ
src/orqviz/__init__.py 100.00% <ø> (ø)
src/orqviz/utils.py 100.00% <ø> (ø)
src/orqviz/loss_function.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@mstechly
Copy link
Contributor Author

I have accidentally commited other things on this branch, closing it and opened #34

@mstechly mstechly closed this Jan 20, 2022
@mstechly mstechly deleted the zqs-920-loss-function-wrapper branch January 20, 2022 16:13
@mstechly mstechly restored the zqs-920-loss-function-wrapper branch January 20, 2022 16:28
@mstechly
Copy link
Contributor Author

I wanted to closed #33 , not this one, sorry for the mess 😅

@mstechly mstechly reopened this Jan 20, 2022
@mstechly mstechly merged commit 2b24039 into dev Jan 21, 2022
@mstechly mstechly deleted the zqs-920-loss-function-wrapper branch January 21, 2022 12:43
AthenaCaesura pushed a commit that referenced this pull request Jan 25, 2023
1ed8593 Chore: spell check the code
188b845 chore: fix spelling errors
3a2857b refactor: remove unused imports
c0a8f7e style: trim trailing whitespace
f7684e5 fix: pin setuptools to fix mypy (#33)
81a6c7f ci: Fix Python exe path in get-next-version (#32)

git-subtree-dir: subtrees/z_quantum_actions
git-subtree-split: 1ed8593c848f60c7416e24ccf2bf2c505178b4b7
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.

2 participants