The proton repo for file common_utils.py

AlibabaResearch / DAMO-ConvAI

DAMO-ConvAI: The official repository which contains the codebase for Alibaba DAMO Conversational AI.

MIT License

1.21k stars 186 forks source link

The proton repo for file common_utils.py #2

Closed liuhuadai closed 2 years ago

liuhuadai commented 2 years ago

Why is “used_schema['column'].add(val1[1])” is val1 instead of val2, I think this place is val2 in LGESQL, what is the meaning of the modification in this place？

huybery commented 2 years ago

Thanks for your question, this bug may come from older version of LGESQL. You can fix it well by following https://github.com/rhythmcao/text2sql-lgesql/commit/ee5bd30f577a9169fc3b0b2f6036e6702dc271ad. Fortunately, the final performance is not affected here