-
Notifications
You must be signed in to change notification settings - Fork 130
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
Further br fixes #2106
Further br fixes #2106
Conversation
if len(cols_as_string) == 0: | ||
return block_on("1") | ||
br: BlockingRuleCreator = CustomRule("1=1", linker._sql_dialect) |
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 type hint was needed for mypy, without it you got an splink/find_brs_with_comparison_counts_below_threshold.py:75: error: Incompatible types in assignment (expression has type "BlockingRuleCreator", variable has type "CustomRule") [assignment]
error
@@ -258,3 +258,19 @@ def count_comparisons_from_blocking_rule_pre_filter_conditions_sqls( | |||
sqls.append({"sql": sql, "output_table_name": "__splink__total_of_block_counts"}) | |||
|
|||
return sqls | |||
|
|||
|
|||
def count_comparisons_from_blocking_rule_pre_filter_conditions( |
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.
Moved out of linker to reduce line count and so it can be more easily called from elsewhere
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.
👍
These fixes apply to the 'beta' (private) auto-blocking functions which I added