`test_transformer.py` failures on NixOS after dependency upgrades

[wariuccio]

Hello, I am looking into these failure I get when building the NixOS package of vegafusion:

FAILED vegafusion-python/tests/test_transformer.py::test_to_arrow_expands_categoricals - assert DataType(large_string) == DataType(string)
FAILED vegafusion-python/tests/test_transformer.py::test_to_table_with_mixed_string_int_column - assert DataType(large_string) == DataType(string)
FAILED vegafusion-python/tests/test_transformer.py::test_to_table_with_all_conversions - assert DataType(large_string) == DataType(string)

The code of Vegafusion we are using is unchanged (still 2.0.3) but there have been recent upgrades to the underlying libraries (e.g. NixOS/nixpkgs@c07b089).

Do you have any advice on how to solve this? Do we really care about the difference between string and large_string or should we loosen the assertion to accept both?

[jonmmease]

Hi @wariuccio, I this may be due to a polars upgrade. Polars has changed default string representations over time. In any case, it's fine to loosen this assertion