audeering / audb

Manage audio and video databases
https://audeering.github.io/audb/
Other
23 stars 1 forks source link

Ensure correct dtype of dependency table #437

Closed hagenw closed 3 months ago

hagenw commented 3 months ago

Closes #436

When loading the dependency table from cache we did only adjust its data types if we detected that the index was not stored as data type object as we switched from string to object when introducing the new pyarrow data types for the dependency table in version 1.7.0 of audb.

But we forgot, that we had used object as data type of the index before, and switched to string only in version 1.6.4 of audb:

image

Which means, if a user has a dependency table in cache stored with audb <1.6.4 when trying to publish a new version of a database, they will run into the issue reported in #436.

To fix it, I updated the code to always apply the data type conversion now. I also checked if it had any implications on execution time (as this was the main motivation for the if clause before), but I was not able to find any change in execution time.