Skip to content

Fix pandas qcut link for the EqualFrequencyDiscretiser user guide - #748

Merged
solegalli merged 1 commit into
feature-engine:mainfrom
cmcouto-silva:UserGuide_EqualFrequencyDiscretiser
Apr 28, 2024
Merged

Fix pandas qcut link for the EqualFrequencyDiscretiser user guide#748
solegalli merged 1 commit into
feature-engine:mainfrom
cmcouto-silva:UserGuide_EqualFrequencyDiscretiser

Conversation

@cmcouto-silva

Copy link
Copy Markdown
Contributor

The current text (Pandas cut) links to the Pandas cut library. I am just replacing cut with qcut since that's what EqualFrequencyDiscretiser uses under the hood.

The current text (Pandas cut) is linking to the pandas cut library. I am just replacing cut by qcut, since it's what EqualFrequencyDiscretiser uses under the hood.
@codecov

codecov Bot commented Apr 26, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.18%. Comparing base (a718b17) to head (689f1c8).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #748   +/-   ##
=======================================
  Coverage   98.18%   98.18%           
=======================================
  Files         105      105           
  Lines        4074     4074           
  Branches      795      795           
=======================================
  Hits         4000     4000           
  Misses         29       29           
  Partials       45       45           

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

@solegalli
solegalli merged commit 8e53f03 into feature-engine:main Apr 28, 2024
@solegalli

Copy link
Copy Markdown
Collaborator

Thank you!

solegalli pushed a commit that referenced this pull request May 6, 2024
The current text (Pandas cut) is linking to the pandas cut library. I am just replacing cut by qcut, since it's what EqualFrequencyDiscretiser uses under the hood.
@cmcouto-silva
cmcouto-silva deleted the UserGuide_EqualFrequencyDiscretiser branch July 6, 2024 15:39
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