search

audeering / audformat

Format to store media files and annotations
https://audeering.github.io/audformat/
Other
11 stars 1 forks source link

Store tables per default as parquet files #452

Closed hagenw closed 4 months ago

hagenw commented 4 months ago

As we now support storing tables as parquet files in audb>=1.8.0, and have published the first databases, I would propose to switch to parquet for storing tables in the next audformat release.

This pull request changes the default values of the storage_format argument in audformat.Database.save() and audformat.Table.save() to "parquet".

image

image

hagenw commented 4 months ago

Seems our tests were already well prepared for this step, as I didn't had to change anything.

hagenw commented 4 months ago

So my understanding is that one could get away without defaulting to parquet in Table - still it is better at the level of API consistency.

Exactly, we don't have to change it in audformat.Table.save(), but I also thought that it would be more consistent this way.