Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Question about table representation for tasks like NL2SQL #16

Open
xiang-deng opened this issue Sep 30, 2020 · 2 comments
Open

Question about table representation for tasks like NL2SQL #16

xiang-deng opened this issue Sep 30, 2020 · 2 comments

Comments

@xiang-deng
Copy link

@pcyin Hi, I have two questions about the spider experiments.

  1. In text2sql for DB with multiple tables, we will also need representation for each table other than columns. In the paper you mentioned the table representation is obtained via the prefix [CLS] token, do you consider the table name in this step? Right now the input to tabert does not have the table name/caption part, but did you concatenate the table name and column name? Otherwise, it seems hard to differentiate columns in different tables with the same name, e.g., name column in tables actor and director.

  2. For the [CLS] representation, is it part of the context_encoding returned by the model as shown in the example? Seems the [CLS] token is considered as part of the context.

It would be very helpful if you could help clarify this. Thanks!

@mellahysf
Copy link

Any responses for that @xiang-deng?

Thanks

@pr3martins
Copy link

I also had the same questions. Have you got any responses @xiang-deng ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants