Skip to content

*python/doc/source/reference: Fix epsilon description#48

Merged
MLopez-Ibanez merged 1 commit intomulti-objective:mainfrom
leandrolanzieri:pr/python/doc/fix_epsilon_formula
Dec 19, 2025
Merged

*python/doc/source/reference: Fix epsilon description#48
MLopez-Ibanez merged 1 commit intomulti-objective:mainfrom
leandrolanzieri:pr/python/doc/fix_epsilon_formula

Conversation

@leandrolanzieri
Copy link
Copy Markdown
Contributor

Hi, thanks for this cool project!

I found a small inconsistency in the epsilon metric while going through the docs. I believe this is what was meant.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.22%. Comparing base (e006780) to head (f64c8ff).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
- Coverage   77.23%   77.22%   -0.02%     
==========================================
  Files          62       62              
  Lines        5786     5786              
  Branches      878      878              
==========================================
- Hits         4469     4468       -1     
- Misses       1126     1127       +1     
  Partials      191      191              
Flag Coverage Δ
R 70.15% <ø> (ø)
python 75.34% <ø> (-0.17%) ⬇️

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

Components Coverage Δ
tests 100.00% <ø> (ø)
src 87.41% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MLopez-Ibanez
Copy link
Copy Markdown
Contributor

You are completely right. Would you mind also fixing it here? https://github.com/multi-objective/moocore/blob/main/r/R/epsilon.R? I'd like to keep the Python and R docs in sync.

@leandrolanzieri
Copy link
Copy Markdown
Contributor Author

Sure! I found also another occurrence in r/man.

@leandrolanzieri leandrolanzieri force-pushed the pr/python/doc/fix_epsilon_formula branch from 906800f to 16c3b6f Compare December 18, 2025 19:54
@MLopez-Ibanez
Copy link
Copy Markdown
Contributor

Thank you! Would you mind squashing the three commits into one? Keeps the history simpler. I can do it myself but it would alter the authorship of the commit.

…ption

r/R/epsilon.R: Fix epsilon description

r/man/epsilon.Rd: Fix epsilon description
@leandrolanzieri leandrolanzieri force-pushed the pr/python/doc/fix_epsilon_formula branch from 16c3b6f to f64c8ff Compare December 19, 2025 12:41
@MLopez-Ibanez MLopez-Ibanez merged commit eba446d into multi-objective:main Dec 19, 2025
28 of 29 checks passed
@MLopez-Ibanez
Copy link
Copy Markdown
Contributor

Thank you!

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