Update lockfile + fixes for latest package versions #2514
Merged
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.
Upgrade packages in lockfiles, and add relevant fixes.
Splink changes:
duckdb
timestamp format to be UTC rather than arbitrary time-zone, as the latter only worked accidentally (duckdb
doesn't accept'Z'
as a timezone specifier for UTC, so existing timestamp formats we used in tests did not work). This shouldn't affect functionality in cases with timestamps coded in ISO-8601 UTC standard, but will mean that anyone using other timezones may need to specify explicit timestamp formats rather than relying on defaults.sqlglot
dialect, preventingsize
function being incorrectly transpiled in more recent package versionsTest-only changes:
substring
instead ofsubstr
, assqlglot
seems to now standardise to the formerDatabaseAPI._sql_to_splink_dataframe(...)
, explicitly register the dummy frame in the connexion. Previouslyduckdb
could still access as it was defined in the same module, but something has changed in howduckdb
looks up python objects, so we need to explicitly register it on every call to ensure we can look up the frame in the backend. We need to do it every call, as actions in the test can drop the physical table (as the nature of the mock means that Splink thinks it created this table, even if it is pre-registered)Closes #2491.
Closes #2511.
For extra constraints in dev dependencies see #2518.