-
Notifications
You must be signed in to change notification settings - Fork 179
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
[enhancement] add sklearnex version of validate_data
, _check_sample_weight
#2177
Merged
icfaust
merged 147 commits into
uxlfoundation:main
from
icfaust:dev/sklearnex_assert_all_finite
Dec 10, 2024
Merged
Changes from 138 commits
Commits
Show all changes
147 commits
Select commit
Hold shift + click to select a range
32fe269
add finiteness_checker pybind11 bindings
icfaust cdbf1b5
added finiteness checker
icfaust 62674a2
Update finiteness_checker.cpp
icfaust c75c23b
Update finiteness_checker.cpp
icfaust 6a20938
Update finiteness_checker.cpp
icfaust 382d7a1
Update finiteness_checker.cpp
icfaust c8ffd9c
Update finiteness_checker.cpp
icfaust 9aa13d5
Update finiteness_checker.cpp
icfaust 84e15d5
Rename finiteness_checker.cpp to finiteness_checker.cpp
icfaust 63073c6
Update finiteness_checker.cpp
icfaust d915da5
Merge branch 'intel:main' into dev/new_assert_all_fininte
icfaust 3dddf2d
add next step
icfaust 1e1213e
follow conventions
icfaust 0531713
make xtable explicit
icfaust e831167
remove comment
icfaust d6eb1d0
Update validation.py
icfaust fb30d6e
Update __init__.py
icfaust 63a18c2
Update validation.py
icfaust 76c0856
Update __init__.py
icfaust 7deb2bb
Update __init__.py
icfaust ed46b29
Update validation.py
icfaust 67d6273
Update _data_conversion.py
icfaust 054f0a1
Merge branch 'main' into dev/new_assert_all_fininte
icfaust 8abead9
Update _data_conversion.py
icfaust 47d0f8b
Update policy_common.cpp
icfaust e48c2bd
Update policy_common.cpp
icfaust c6751c4
Update _policy.py
icfaust f3e4a3a
Update policy_common.cpp
icfaust 39cdb5f
Rename finiteness_checker.cpp to finiteness_checker.cpp
icfaust 0f39613
Create finiteness_checker.py
icfaust b42cfe3
Update validation.py
icfaust 0ed615e
Update __init__.py
icfaust f101aff
attempt at fixing circular imports again
icfaust 24c0e94
fix isort
icfaust 3f96166
remove __init__ changes
icfaust d985053
last move
icfaust 90ec48b
Update policy_common.cpp
icfaust 8c2c854
Update policy_common.cpp
icfaust 6fa38d7
Update policy_common.cpp
icfaust 9c1ca9c
Update policy_common.cpp
icfaust 4b67dbd
Update validation.py
icfaust fa59a3c
add testing
icfaust 3330b33
isort
icfaust 4895940
attempt to fix module error
icfaust 0c6dd5d
add fptype
icfaust e2182fa
fix typo
icfaust 982ef2c
Update validation.py
icfaust 2fb52a8
remove sua_ifcae from to_table
icfaust 28dc267
isort and black
icfaust 2f85fd4
Update test_memory_usage.py
icfaust 8659248
format
icfaust 3827d6f
Update _data_conversion.py
icfaust 55fa7d2
Update _data_conversion.py
icfaust 175cd78
Update test_validation.py
icfaust 7016ad0
remove unnecessary code
icfaust 1a01859
Merge branch 'main' into dev/new_assert_all_fininte
icfaust 2fbcdd9
merge master
icfaust fb7375f
make reviewer changes
icfaust 30816bf
make dtype check change
icfaust abb3b16
add sparse testing
icfaust 97aef73
try again
icfaust 6e29651
try again
icfaust 59363a8
try again
icfaust 12de703
temporary commit
icfaust 07ec3d8
first attempt
icfaust 32c565d
missing change?
icfaust a571a4e
Merge branch 'intel:main' into dev/sklearnex_assert_all_finite
icfaust 5093ed7
modify DummyEstimator for testing
icfaust f04deba
generalize DummyEstimator
icfaust 740a5e7
switch test
icfaust 27050bd
further testing changes
icfaust 53c8f7b
add initial validate_data test, will be refactored
icfaust 90f59c4
fixes for CI
icfaust 7f170e2
Update validation.py
icfaust 81e2bbc
Update validation.py
icfaust 116bdba
Update test_memory_usage.py
icfaust 076ebc4
Update base.py
icfaust e1d0743
Update base.py
icfaust f59cdd3
improve tests
icfaust 7f9ea25
fix logic
icfaust 51247c0
fix logic
icfaust 6e5c0ef
fix logic again
icfaust 8d47744
rename file
icfaust 1ae9af5
Revert "rename file"
icfaust bf9b46e
remove duplication
icfaust 3101c3f
fix imports
icfaust 6da176b
Merge branch 'intel:main' into dev/sklearnex_assert_all_finite
icfaust ee799f6
Rename test_finite.py to test_validation.py
icfaust db4a6c6
Revert "Rename test_finite.py to test_validation.py"
icfaust b5acbac
updates
icfaust ed57c15
Update validation.py
icfaust 414f897
fixes for some test failures
icfaust 83253b3
fix text
icfaust b22e23a
fixes for some failures
icfaust 2f8ec16
make consistent
icfaust 1fd9973
fix bad logic
icfaust c20c8cc
fix in string
icfaust 1ce1b10
attempt tp see if dataframe conversion is causing the issue
icfaust 5355039
fix iter problem
icfaust b5b8442
fix testing issues
icfaust d025c89
formatting
icfaust 428bfb6
revert change
icfaust da23138
fixes for pandas
icfaust 1d0c330
there is a slowdown with pandas that needs to be solved
icfaust f3f63a6
swap to transpose for speed
icfaust 56c8054
more clarity
icfaust 1580d77
add _check_sample_weight
icfaust ffc9f1f
add more testing'
icfaust d184ed0
rename
icfaust c68616f
remove unnecessary imports
icfaust e7ea94e
fix test slowness
icfaust dbe108d
focus get_dataframes_and_queues
icfaust 7284b59
put config_context around
icfaust e1be91d
Update test_validation.py
icfaust 8a0f9e9
Update base.py
icfaust 5272207
Update test_validation.py
icfaust 21a7896
Merge branch 'intel:main' into dev/sklearnex_assert_all_finite
icfaust 56b5c4c
generalize regex
icfaust 0d1b306
add fixes for sklearn 1.0 and input_name
icfaust 8ff312e
fixes for test failures
icfaust 87b7e3b
Update validation.py
icfaust 29e8f8c
Update test_validation.py
icfaust 527ce22
Merge branch 'intel:main' into dev/sklearnex_assert_all_finite
icfaust 27ce5fc
Update validation.py
icfaust 5d31988
formattintg
icfaust c4dccd6
make suggested changes
icfaust f83f1ef
follow changes made in #2126
icfaust 0356a90
Merge branch 'intel:main' into dev/sklearnex_assert_all_finite
icfaust e43c047
fix future device problem
icfaust a9504a8
Merge branch 'dev/sklearnex_assert_all_finite' of https://github.com/…
icfaust b799d44
Merge branch 'intel:main' into dev/sklearnex_assert_all_finite
icfaust 5c81f9d
Update validation.py
icfaust 6ef96b1
merge main
icfaust 1db7575
Merge branch 'uxlfoundation:main' into dev/sklearnex_assert_all_finite
icfaust 8fca003
Merge branch 'uxlfoundation:main' into dev/sklearnex_assert_all_finite
icfaust 164435d
minor changes based on #2206, suggestions
icfaust 4aff9e0
remove xp as keyword
icfaust db11608
only_non_negative -> ensure_non_negative
icfaust bf62e50
add commentary
icfaust 993a272
formatting
icfaust c034883
address changes
icfaust e5c9b8b
Update test_validation.py
icfaust c55843b
Update base.py
icfaust ac6d831
Update test_validation.py
icfaust ebd282a
Merge branch 'uxlfoundation:main' into dev/sklearnex_assert_all_finite
icfaust 0ef3971
Merge branch 'uxlfoundation:main' into dev/sklearnex_assert_all_finite
icfaust 1305ca1
Update sklearnex/utils/validation.py
icfaust File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it Ok that
y_attr_
goes with X's queue?Also, what happens if
X
andy
are from different namespaces and have differentsua_iface
? For example,X
- from dpnp andy
- from numpy. What is expected to happen in this case?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a failure in the original implementation, though the backend in sklearnex is very fuzzy in this (not standard). For questions about from_table I would ask @samir-nasibli.