gen-arg icon indicating copy to clipboard operation
gen-arg copied to clipboard

the implementation part of clarification in code?

Open crescent73 opened this issue 2 years ago • 3 comments

Hello, I found that in the “clarification” methods mentioned in Section 2.2 of the paper, I did not see the implementation in the code?

crescent73 avatar Nov 10 '22 02:11 crescent73

Hi Cresent73, It is true that the current code base does not cover the entity type statement. It was originally implemented as a postprocessing script that takes multiple decoding results and reranks them based on the probability of the decoding result + type statement. I'm working on another deadline now and will add more details on this over the weekend.

raspberryice avatar Nov 10 '22 07:11 raspberryice

Thanks for you reply. I still wonder that the clarification is only implemented by postprocessing script? I think it was a part of the model output(argument + type), so model can learn the type information during train.Is there something wrong with my understanding? Thanks again, and looing forward to your details explationation and implementation.

crescent73 avatar Nov 10 '22 08:11 crescent73

It's generally not a good idea to make BART output very long sequences (if you think of concatenating the filled templated with all of the type statements). One workaround is to simply modify the templates to make the argument types more explicit. I think this is what they did in this paper: https://arxiv.org/pdf/2108.12724.pdf

raspberryice avatar Nov 10 '22 17:11 raspberryice