Allow dynamic CSV column names in add_ph_num.py

openvpi / MakeDiffSinger

Pipelines and tools to build your own DiffSinger dataset.

BSD 3-Clause "New" or "Revised" License

87 stars 23 forks source link

Closed qiuqiao closed 5 months ago

qiuqiao commented 5 months ago

对于原先的代码，如果输入的transcriptions.csv的columns包含除了name、ph_seq、ph_dur以外的其他column，那么add_ph_num会报错。sofa直出的transcriptions.csv带有word_seq与word_dur，不能直接使用，比较麻烦。

此pr改了一行代码，可以兼容带有各种columns、各种顺序columns的transcriptions.csv。

yqzhishen commented 5 months ago

我可以合并这个PR，但需要提醒的是，这并不意味着往CSV里添加未定义的列是合理的

qiuqiao commented 5 months ago

我可以合并这个PR，但需要提醒的是，这并不意味着往CSV里添加未定义的列是合理的

好的。也许我会考虑sofa导出时去除csv里未定义的列。