model_cls_wrapper is not used

LUMIA-Group / rasat

The official implementation of the paper "RASAT: Integrating Relational Structures into Pretrained Seq2Seq Model for Text-to-SQL"(EMNLP 2022)

https://arxiv.org/abs/2205.06983

Apache License 2.0

63 stars 18 forks source link

model_cls_wrapper is not used #6

Closed Dongfeng-He closed 1 year ago

Dongfeng-He commented 1 year ago

Hi, @JiexingQi, I notice that model_cls_wrapper is not used when PICARD is enable, which is different from the implementation of PICARD. Will PICARD work if we don't wrap the model with model_cls_wrapper?

The code references are listed below: RASAT implementation: https://github.com/LUMIA-Group/rasat/blob/main/seq2seq/run_seq2seq.py#L170 PICARD implementation: https://github.com/ServiceNow/picard/blob/main/seq2seq/run_seq2seq.py#L162

Dongfeng-He commented 1 year ago

The model_cls_wrapper is used in eval_run_seq2seq.py.