Multi-View-Seq2Seq icon indicating copy to clipboard operation
Multi-View-Seq2Seq copied to clipboard

unintended bug on sep (Read_Labels.ipynb)

Open leadawon opened this issue 2 years ago • 0 comments

Hello, I'm leadawon. I'm a beginner in nlp.

It's been a long time since you published your paper. I think it is great research!

I think I found a bug, which baffled me.

I ran Read_Labels.ipynb to create *.source. When executing

cons, sum = concat_conversion(data, labels, label_type)

within the

transform_format(prefix, label_type = '_sent_c99_label') function

, the label_type goes into sep field in concat_conversion(data, labels, sep=' |', label_type = '_sent_c99_label') function. If you think this is a problem, I would appreciate it if you could fix it. I'm always rooting for you!


my sol. cons, sums = concat_conversation(data, labels, label_type) -> cons, sums = concat_conversation(data, labels, " | " ,label_type)

leadawon avatar May 29 '23 01:05 leadawon