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

Allow Optimization with OptimizableCollections to Work #857

Closed
wants to merge 6 commits into from

remove unneeded line in utils

366fd43
Select commit
Loading
Failed to load commit list.
Closed

Allow Optimization with OptimizableCollections to Work #857

remove unneeded line in utils
366fd43
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Feb 6, 2024 in 1s

94.87% (-0.05%) compared to dc7d2eb

View this Pull Request on Codecov

94.87% (-0.05%) compared to dc7d2eb

Details

Codecov Report

Attention: 17 lines in your changes are missing coverage. Please review.

Comparison is base (dc7d2eb) 94.92% compared to head (366fd43) 94.87%.
Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #857      +/-   ##
==========================================
- Coverage   94.92%   94.87%   -0.05%     
==========================================
  Files          80       80              
  Lines       19619    19724     +105     
==========================================
+ Hits        18623    18713      +90     
- Misses        996     1011      +15     
Files Coverage Δ
desc/magnetic_fields.py 96.08% <100.00%> (+0.01%) ⬆️
desc/objectives/__init__.py 100.00% <ø> (ø)
desc/objectives/utils.py 97.14% <93.33%> (-0.76%) ⬇️
desc/optimizable.py 96.80% <80.00%> (-1.00%) ⬇️
desc/objectives/_geometry.py 97.72% <90.90%> (-0.71%) ⬇️
desc/objectives/linear_objectives.py 95.73% <84.41%> (-0.92%) ⬇️

... and 1 file with indirect coverage changes