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

Moving Rhino independent files to Tunny.Core #250

Merged
merged 7 commits into from
Feb 20, 2024
Merged

Conversation

hrntsm
Copy link
Owner

@hrntsm hrntsm commented Feb 20, 2024

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Moving Rhino independent files to Tunny.Core

@hrntsm hrntsm self-assigned this Feb 20, 2024
@hrntsm hrntsm added the enhancement New feature or request label Feb 20, 2024
Copy link

codeclimate bot commented Feb 20, 2024

Code Climate has analyzed commit c674cfe and detected 12 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 4
Duplication 8

View more on Code Climate.

Copy link

codecov bot commented Feb 20, 2024

Codecov Report

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

Comparison is base (25e544b) 70.54% compared to head (c674cfe) 47.63%.
Report is 1 commits behind head on develop.

Files Patch % Lines
Tunny.Core/Settings/Sampler/SamplerSettings.cs 21.62% 29 Missing ⚠️
Tunny.Core/Settings/Sampler/CmaEs.cs 30.76% 27 Missing ⚠️
Tunny.Core/Settings/Sampler/NSGAII.cs 20.00% 24 Missing ⚠️
Tunny.Core/Settings/Sampler/Tpe.cs 39.28% 17 Missing ⚠️
Tunny.Core/Settings/Sampler/NSGAIII.cs 14.28% 12 Missing ⚠️
Tunny.Core/Settings/Sampler/QMC.cs 0.00% 9 Missing ⚠️
Tunny.Core/Settings/Sampler/BoTorch.cs 20.00% 8 Missing ⚠️
Tunny.Core/Settings/Sampler/Random.cs 0.00% 6 Missing ⚠️
Tunny.Core/Settings/Storage.cs 0.00% 3 Missing ⚠️
Tunny.Core/Storage/JournalStorage.cs 0.00% 3 Missing ⚠️
... and 6 more
Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #250       +/-   ##
============================================
- Coverage    70.54%   47.63%   -22.92%     
============================================
  Files            7       34       +27     
  Lines         1015     1688      +673     
  Branches        78      133       +55     
============================================
+ Hits           716      804       +88     
- Misses         275      859      +584     
- Partials        24       25        +1     

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

@hrntsm hrntsm merged commit d22bfef into develop Feb 20, 2024
5 of 9 checks passed
@hrntsm hrntsm deleted the featura/move-tunny-core branch February 20, 2024 02:42
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.

1 participant