#264 v3.2: Import Annotation Layer

[Niranjan Sarkar]

From version 3.2 onwards, importing the annotation layer using CSV defaults to Time Instance . Earlier it was Time Value layer. I now miss to add the label/Value/Time columns. Or have I missed something?
Thanks.

[Chris Cannam]

Thanks for the report - I'll have a look when I get back to work in the new year.

In the mean time, do you have an example CSV file that you could attach (or send to me) along with a description of how to produce the new and undesirable behaviour?

[Niranjan Sarkar]

Hi Chris,
Thanks for your response. I have observed something that explains the behavior. From v3.2 the "|" symbol is treated as a valid seperator. Earlier it was "ignored". I think this is causing the parser to behave in this manner. When I escaped it with double quotes, I get back the original behavior. Refer to the sample csv attached, and you could check by ecnlosing the first column with double quotes to get back the original behavior of v3.1.1. In a way v3.2 breaks backwards compatibility. For me it is not a big issue, I am the onwner of the code that is generating my csv, so I can easily modify. For other cases it might be a problem. Best regards, Niranjan

[Chris Cannam]

Thanks for the extra information - I see what you mean. The CSV logic saw some updates between 3.1.1 and 3.2 to make it more consistent in a number of ways, but this is one pitfall I hadn't seen. I'll have to see about adding an option to choose the delimiter.