forked from rasbt/mlxtend
-
Notifications
You must be signed in to change notification settings - Fork 0
[pull] master from rasbt:master #9
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
Open
pull
wants to merge
429
commits into
vishalbelsare:master
Choose a base branch
from
rasbt:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+21,043
−14,151
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* updated dependency on distutils for python 3.12 and above * removed unused import
* update changelog * update 2024
Tests should be included in the source distribution but there is no reason to include them in the build wheel. Disable implicit packages (directories without an __init__.py file) altogether by setting the following in pyproject.toml: [tool.setuptools.packages.find] include = ["mlxtend", "mlxtend.*"] namespaces = false I assume the files in mlxtend/data/data are still needed so I've modified the package data to include files in that directory.
* update ci * black update * conda init * fix workflow * fix workflow * fix workflow * skip tensorflow dependency * formatting
* modified: mlxtend/classifier/tests/test_stacking_classifier.py - Marked :test:`test_StackingClassifier` as skip because scikit-learn implemented its own :class:`StackingClassifier` in 0.22. * modified: mlxtend/classifier/tests/test_stacking_classifier.py - Formatted with isort, black, flake8. * modified: mlxtend/classifier/tests/test_stacking_classifier.py - Skipping all failing unit tests related to :class:`StackingClassifier` as they don't align with `scikit-learn`'s implementation. * modified: mlxtend/classifier/tests/test_stacking_cv_classifier.py - Skipping failed unit tests because scikit-learn's StackingClassifier has built-in cross-validation support. * modified: .github/workflows/python-package-conda.yml - Updated scikit-learn version to 1.3.1 modified: environment.yml - Updated scikit-learn version to 1.3.1 modified: requirements.txt - Updated scikit-learn version to 1.3.1 * modified: mlxtend/preprocessing/tests/test_transactionencoder.py - Updated failing unit test to compare output directly instead of converting to numpy arrays (which results in errors unless the dtype is set to object). * modified: mlxtend/regressor/tests/test_stacking_cv_regression.py - Skipping `test_gridsearch_replace_mix` as it uses `StackingCVRegressor` when `scikit-learn` has its own implementation as of 0.22. * fix stacking classifiers * update * adjust for macos vs linux precision * autoformat workflow * autoformat workflow * hopefully final update * hopefully final update * leeway for linux * leeway for linux --------- Co-authored-by: rasbt <[email protected]>
#1087) * modified: test_transactionencoder.py - Added two new tests, `test_get_feature_names_out` and `test_set_output`. Passing these tests is a step towards the output of `TransactionEncoder` being formatted as a pandas.DataFramed by default. * modified: transactionencoder.py - Added `get_feature_names_out` method to `TransactionEncoder` to expose the `set_output` method. * modified: tests/test_transactionencoder.py - Updated test to include more checks. It is now back in a failing state. * modified: tests/test_transactionencoder.py - Updated test_set_output docstring to be more explicit. - Added numpy assertion to check that the transformed output columns match the original columns_ attribute for test_set_output. - Added numpy assertion to check that the get_feature_names_out output match the original columns_ attribute for test_get_feature_names_out. * modified: transactionencoder.py - Added logic similar to that in `sklearn.base.ClassNamePrefixFeaturesOutMixin` and `sklearn.base.OneToOneFeatureMixin` for the get_feature_names_out method. * modified: docs/sources/user_guide/preprocessing/TransactionEncoder.ipynb - Updated the user guide to show both the get_feature_names_out method and the set_output method. * modified: docs/sources/CHANGELOG.md - Updated changelog to reflect new features. * modified: docs/sources/CHANGELOG.md - Updated issue number. * modified: docs/sources/CHANGELOG.md - Updated issue number (again) to reflect the PR link instead of the issue link. * modified: mlxtend/preprocessing/transactionencoder.py - Ran isort over imports to fix failing check in PR. * modified: requirements.txt - Increased scikit-learn version to minimum required for set_output to work. * modified: environment.yml - Bumped scikit-learn version up to 1.2.2 to match requirements.txt. * modified: .github/workflows/python-package-conda.yml - Bumped scikit-learn version up to 1.2.2 to match environment.yml and requirements.txt. * modified: mlxtend/preprocessing/tests/test_transactionencoder.py - Updated `test_inverse_transform` to passing state by removing conversion to numpy array. * modified: .github/workflows/python-package-conda.yml - Updated scikit-learn version to 1.3.1 to integerate fix from scikit-learn/scikit-learn#27044 modified: environment.yml - Updated scikit-learn version to 1.3.1 to integerate fix from scikit-learn/scikit-learn#27044 modified: requirements.txt - Updated scikit-learn version to 1.3.1 to integerate fix from scikit-learn/scikit-learn#27044 * Update mlxtend/preprocessing/transactionencoder.py * Update mlxtend/preprocessing/transactionencoder.py * Update mlxtend/preprocessing/transactionencoder.py --------- Co-authored-by: Sebastian Raschka <[email protected]>
Co-authored-by: Sebastian Raschka <[email protected]>
* Update minor typos in fpgrowth.ipynb Fixing minor typos in the markdown of notebooks for the FP-Growth webpage * fix merge conflict --------- Co-authored-by: Sebastian Raschka <[email protected]>
* Replace numpy with scipy decomps. * add changelog entry --------- Co-authored-by: Sebastian Raschka <[email protected]>
* Adding the code for calculating Jaccard Similiarity * Added some jaccard tests * More jaccard tests * Implemented Centered Confidence and Certainty Factor * Added parameter to choose only certain metrics for association_rules * removed centered confidence * remove print statements in test * Implemented kulczynski algorithm Nachiket Deo <[email protected]> * Implemented Kulcynski + Certainty tests Co-authored-by: Nachiket18 <[email protected]> Co-authored-by: AnkithN <[email protected]> * fixed flake8 formatting * Added jaccard, certainty, and kulczynski to notebook documentation Co-authored-by: AnkithN <[email protected]> * Added changes to changelog * Update test_association_rules.py * Update test_association_rules.py * Update association_rules.py * Update CHANGELOG.md --------- Co-authored-by: Nachiket18 <[email protected]> Co-authored-by: ankithn30 <[email protected]> Co-authored-by: AnkithN <[email protected]> Co-authored-by: Nachiket18 <[email protected]> Co-authored-by: AnkithN <[email protected]> Co-authored-by: Sebastian Raschka <[email protected]>
* finalize_fit() support for numpy >= 2.0 * updated changelog * flake8: removed blank line * flake8: removed trailing whitespaces * fixed changelog indents * fix for isort / black formatter * added change/feature description
* added sklearn version compability for fit_params * added change to changelog * fixed format * fixed order of imports * formatted with isort and black
* updated CI/CD workflows * Update .github/workflows/python-package-conda.yml --------- Co-authored-by: Sebastian Raschka <[email protected]>
* fixed test_end_idx calculation * added legend title * fixed intersection of xticklabels for Sample index * added changes to changelog * Update CHANGELOG.md * removed redundancy * start x-axis at the origin of coordinates * fixed CV iteration labels * revert idx change * Update CHANGELOG.md * add groups legend * fixed cmap in groups legend
* Update exhaustive_feature_selector.py AttributeError: `np.NINF` was removed in the NumPy 2.0 release. Use `-np.inf` instead. Fixed this Error. * Update mlxtend/feature_selection/exhaustive_feature_selector.py Co-authored-by: Peter Gedeck <[email protected]> --------- Co-authored-by: Sebastian Raschka <[email protected]> Co-authored-by: Peter Gedeck <[email protected]>
* Updated FPGrowth/FPMax and Association Rules with the existence of missing values * Re-structure and document code * Update unit tests * Update CHANGELOG.md * Modify the corresponding documentation in Jupyter notebooks * Final modifications * Fix association rules and corresponding tests * Fix typos * Fixing memory usage increase * Fixing memory usage increase --------- Co-authored-by: Sebastian Raschka <[email protected]>
* 0.23.4 release prep * 0.23.4 release prep
* fix: update parameter handling for sklearn version compatibility in stacking_cv * added fix to changelog * removed trailing whitespaces for flake8
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )