ADBond / splinkclickhouse

Allows Clickhouse to be used as the execution engine for Splink
MIT License
5 stars 0 forks source link

Some more tests #16

Closed ADBond closed 2 months ago

ADBond commented 2 months ago

Again, mainly to check clickhouse tests in CI (#15).

Issue arose with unlinkables test, whose origin was (partially) to do with NULL values being converted to the empty string '' in chdb (see #17). Introduced a workaround in tests for this, so that we can treat empty strings as though they were NULL.

This manifested as an error with unlinkables chart when we only have match_probability values that round to 1, which is an issue upstream.