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

test: improve test coverage #78

Merged
merged 7 commits into from
Feb 15, 2023
Merged

Conversation

Benjamin-eecs
Copy link
Member

Description

  • Improve code coverage
  • Update README
  • Add codecov.yml

@codecov-commenter
Copy link

codecov-commenter commented Sep 13, 2022

Codecov Report

Base: 70.36% // Head: 91.73% // Increases project coverage by +21.37% 🎉

Coverage data is based on head (fa1d6a3) compared to base (c58dff7).
Patch coverage: 97.61% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #78       +/-   ##
===========================================
+ Coverage   70.36%   91.73%   +21.37%     
===========================================
  Files          72       66        -6     
  Lines        3009     2551      -458     
===========================================
+ Hits         2117     2340      +223     
+ Misses        892      211      -681     
Flag Coverage Δ
unittests 91.73% <97.61%> (+21.37%) ⬆️

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

Impacted Files Coverage Δ
torchopt/__init__.py 100.00% <ø> (ø)
torchopt/pytree.py 100.00% <ø> (+28.99%) ⬆️
torchopt/utils.py 86.77% <77.78%> (+29.92%) ⬆️
torchopt/transform/utils.py 91.07% <81.82%> (+16.07%) ⬆️
torchopt/base.py 86.36% <83.33%> (+8.75%) ⬆️
torchopt/alias/utils.py 98.77% <97.87%> (+5.91%) ⬆️
torchopt/accelerated_op/__init__.py 100.00% <100.00%> (+77.27%) ⬆️
torchopt/accelerated_op/_src/adam_op.py 100.00% <100.00%> (+100.00%) ⬆️
torchopt/accelerated_op/adam_op.py 100.00% <100.00%> (+25.32%) ⬆️
torchopt/alias/adam.py 100.00% <100.00%> (+21.74%) ⬆️
... and 55 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Benjamin-eecs Benjamin-eecs added the enhancement New feature or request label Sep 13, 2022
@Benjamin-eecs Benjamin-eecs self-assigned this Sep 16, 2022
@XuehaiPan XuehaiPan force-pushed the fix/test_coverage branch 2 times, most recently from 056877c to c0da995 Compare October 13, 2022 11:28
@XuehaiPan XuehaiPan changed the title fix: improve test coverage test: improve test coverage Nov 15, 2022
@Benjamin-eecs Benjamin-eecs marked this pull request as draft November 15, 2022 21:17
@Benjamin-eecs Benjamin-eecs marked this pull request as ready for review January 17, 2023 10:10
@Benjamin-eecs Benjamin-eecs added this to the 0.7.0 milestone Jan 19, 2023
@XuehaiPan XuehaiPan force-pushed the fix/test_coverage branch 7 times, most recently from 87275d3 to 7737f70 Compare February 5, 2023 06:03
@XuehaiPan XuehaiPan force-pushed the fix/test_coverage branch 4 times, most recently from 89f8d9e to 1c5237e Compare February 12, 2023 12:58
@Benjamin-eecs Benjamin-eecs removed the request for review from JieRen98 February 14, 2023 04:54
@XuehaiPan XuehaiPan force-pushed the fix/test_coverage branch 8 times, most recently from 37d6dde to 98cc0b1 Compare February 15, 2023 08:07
@XuehaiPan XuehaiPan added bug Something isn't working cxx / cuda Something related to the CXX / CUDA extension labels Feb 15, 2023
@XuehaiPan XuehaiPan merged commit c67476b into metaopt:main Feb 15, 2023
@Benjamin-eecs Benjamin-eecs deleted the fix/test_coverage branch February 15, 2023 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cxx / cuda Something related to the CXX / CUDA extension enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants