Bailin

are they supposed to be the same thing? i.e., " desc['column_to_table'][str(col)]" is actually the "table"

We tuned the bert model very carefully with a very small lr (3e-6), but the price to pay is that it has to be trained for much longer. I guess...

Hi, @DevanshChoubey I don't think that's the reason. Sorry for the confusion. There seems to be only one or two sqls in Spider that is not well-structured. The patch we...

@alexpolozov I see. I guess we could add the script https://github.com/rshin/seq2struct/blob/master/data/spider-20190205/generate.sh and the patch file to this repo, as Richard did. I could help submit a PR to fix this.

hi, @zhangyuchen584 In the paper we use dropout 0.21 ~=0.2068, encoder dropout is only used for non-bert model (encoderbert does not have this option). Which attention are you referring to?

hi, all I've attached my original model config below which is used for evaluation (V3) on Spider leaderboard. After careful comparison with my original config, I think I might missed...

Can you try set 'eval_use_heuristic' to False https://github.com/microsoft/rat-sql/blob/f2e00333d425b3bb3b625a89f77f88d015553a6f/experiments/wikisql-glove-run.jsonnet#L12

I think they've updated the dataset.