Skip to content

Comments

fix: incorrect type hint for number_of_bins parameter#567

Merged
lars-reimann merged 1 commit intomainfrom
fix-type
Mar 4, 2024
Merged

fix: incorrect type hint for number_of_bins parameter#567
lars-reimann merged 1 commit intomainfrom
fix-type

Conversation

@lars-reimann
Copy link
Member

@lars-reimann lars-reimann commented Mar 4, 2024

Summary of Changes

Change the type of the number_of_bins parameter of Discretizer.__init__ from float to int.

@lars-reimann lars-reimann requested a review from a team as a code owner March 4, 2024 19:44
@github-actions
Copy link
Contributor

github-actions bot commented Mar 4, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 1 0 0 0.62s
✅ PYTHON mypy 1 0 1.73s
✅ PYTHON ruff 1 0 0 0.15s
✅ REPOSITORY git_diff yes no 0.14s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@codecov
Copy link

codecov bot commented Mar 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (b1e308d) to head (ceb2a73).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #567   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           50        50           
  Lines         2866      2866           
=========================================
  Hits          2866      2866           

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

@lars-reimann lars-reimann merged commit b434e53 into main Mar 4, 2024
@lars-reimann lars-reimann deleted the fix-type branch March 4, 2024 19:52
lars-reimann pushed a commit that referenced this pull request Apr 3, 2024
## [0.20.0](v0.19.0...v0.20.0) (2024-04-03)

### Features

* add deterministic hash methods to all types ([#573](#573)) ([f6a3ca7](f6a3ca7))
* add fnn functionality ([#529](#529)) ([ce53153](ce53153)), closes [#522](#522)
* add suffixes to models to indicate their task ([#588](#588)) ([d490dee](d490dee))
* added lag_plot ([#548](#548)) ([0fb38d2](0fb38d2)), closes [#519](#519)
* added normal plot for time series ([#550](#550)) ([dbdf11e](dbdf11e)), closes [#549](#549)
* when using from table to time series feature must be given ([#572](#572)) ([ca23f0f](ca23f0f)), closes [#571](#571)

### Bug Fixes

* incorrect type hint for `number_of_bins` parameter ([#567](#567)) ([b434e53](b434e53))
* mark various API elements as internal ([#587](#587)) ([ea176fc](ea176fc)), closes [#582](#582) [#585](#585)
@lars-reimann
Copy link
Member Author

🎉 This PR is included in version 0.20.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lars-reimann lars-reimann added the released Included in a release label Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Included in a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant