-
Notifications
You must be signed in to change notification settings - Fork 157
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
Encode int column #780
Encode int column #780
Conversation
dataprofiler/tests/profilers/test_numeric_stats_mixin_profile.py
Outdated
Show resolved
Hide resolved
"sturges", | ||
"sqrt", | ||
] | ||
expected_min_histogram_bin = 1000 |
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.
were you having issue cutting this down from 1000? Or decide to keep it at 1000?
def test_json_encode(self): | ||
self.maxDiff = None | ||
|
||
mixin = NumericStatsMixin() |
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.
empty mixin
?
options = IntOptions() | ||
options.histogram_and_quantiles.bin_count_or_method = 5 | ||
profiler = IntColumn("0", options) |
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.
nice! only nit pick suggestion would be
options = IntOptions() | |
options.histogram_and_quantiles.bin_count_or_method = 5 | |
profiler = IntColumn("0", options) | |
int_options = IntOptions() | |
int_options.histogram_and_quantiles.bin_count_or_method = 5 | |
profiler = IntColumn("0", options=int_options) |
@@ -1,3 +1,4 @@ | |||
import json |
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.
When the other PR gets merged, this can get updated and we won't have this code in this one.
Head branch was pushed to by a user without write access
f23ee19
to
11051ad
Compare
Head branch was pushed to by a user without write access
7b6b6b8
to
d6fe67e
Compare
Head branch was pushed to by a user without write access
@@ -25,7 +25,7 @@ def is_in_range(x: float | int, config: dict) -> bool: | |||
raise TypeError("Value is not a float") | |||
|
|||
|
|||
def is_in_list(x: str, config: dict) -> bool: | |||
def is_in_list(x: str | int | float, config: dict) -> bool: | |||
""" | |||
Check to see x is in the config list. | |||
|
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.
make sure to change the docstring with the type updates as well
279077f
to
5aa8f3b
Compare
No description provided.