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

Folder Restructure for OpenQAOA #170

Merged
merged 92 commits into from
Feb 17, 2023
Merged

Folder Restructure for OpenQAOA #170

merged 92 commits into from
Feb 17, 2023

Conversation

shahidee44
Copy link
Collaborator

@shahidee44 shahidee44 commented Jan 25, 2023

Description

  • Restructured folders with OpenQAOA (This is a breaking change!)
  • Updated unittests
  • Updated example notebooks
  • Changed setup.py

Checklist

  • I have performed a self-review of my code.
  • I have commented my code and used numpy-style docstrings
  • I have made corresponding updates to the documentation.
  • My changes generate no new warnings
  • I have added/updated tests to make sure bugfix/feature works.
  • New and existing unit tests pass locally with my changes.

Type of change

Please delete options that are not relevant.

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Name the new unit-tests that you have added along with this change.

test_all.py locally.

@codecov
Copy link

codecov bot commented Jan 25, 2023

Codecov Report

Merging #170 (dcec534) into dev (e9ba397) will increase coverage by 8.22%.
The diff coverage is 100.00%.

❗ Current head dcec534 differs from pull request most recent head 389d57f. Consider uploading reports for the commit 389d57f to get more accurate results

@@            Coverage Diff             @@
##              dev     #170      +/-   ##
==========================================
+ Coverage   81.89%   90.11%   +8.22%     
==========================================
  Files         106       30      -76     
  Lines       12535     5685    -6850     
==========================================
- Hits        10265     5123    -5142     
+ Misses       2270      562    -1708     
Impacted Files Coverage Δ
tests/test_all.py 100.00% <ø> (ø)
tests/test_analytical_simulator.py 100.00% <ø> (ø)
tests/test_backends.py 52.45% <ø> (+6.40%) ⬆️
tests/test_converters.py 100.00% <ø> (ø)
tests/test_derivative.py 100.00% <ø> (ø)
tests/test_gates.py 100.00% <ø> (ø)
tests/test_imports.py 89.47% <ø> (ø)
tests/test_logger.py 100.00% <ø> (ø)
tests/test_notebooks.py 95.55% <ø> (-0.10%) ⬇️
tests/test_operators.py 100.00% <ø> (ø)
... and 105 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@shahidee44 shahidee44 merged commit 5219a09 into dev Feb 17, 2023
@shahidee44 shahidee44 deleted the de_folder_restruct branch February 17, 2023 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants