Skip to content
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

Bug in save model to JSON #2061

Closed
2 tasks done
aymonwuolanne opened this issue Mar 15, 2024 · 0 comments
Closed
2 tasks done

Bug in save model to JSON #2061

aymonwuolanne opened this issue Mar 15, 2024 · 0 comments

Comments

@aymonwuolanne
Copy link
Contributor

What happens?

When running linker.save_model_to_json() the field tf_minimum_u_value is dropped from comparison levels. This is because the implementation of as_dict() for ComparisonLevel doesn't mention it.

To Reproduce

import splink.comparison_level as cl

c = cl.ComparisonLevel({"sql_condition": "name_l = name_r", "label_for_charts": "exact match", "tf_minimum_u_value": 1e-4})

print(c.as_dict())

OS:

Windows (WSL)

Splink version:

3.9.13

Have you tried this on the latest master branch?

  • I agree

Have you tried the steps to reproduce? Do they include all relevant data and configuration? Does the issue you report still appear there?

  • I agree
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant