search

ServiceNow / picard

PICARD - Parsing Incrementally for Constrained Auto-Regressive Decoding from Language Models. PICARD is a ServiceNow Research project that was started at Element AI.
https://arxiv.org/abs/2109.05093
Apache License 2.0
341 stars 122 forks source link

PICARD client launch issue #133

Open Rautmare-Sneha opened 1 year ago

Rautmare-Sneha commented 1 year ago

Hi

I am executing the Eval code on the spider data with picard using docker image

provided 777 permission to mentioned 2 directories 'make eval' inside docker -> Install stanza package python3 seq2seq/eval_run_seq2seq.py configs/spider/eval_spider_rasat_4160.json After preprocessing , generating relation i got following error :

` Use relation model. Traceback (most recent call last): File "seq2seq/eval_run_seq2seq.py", line 309, in main() File "seq2seq/eval_run_seq2seq.py", line 197, in main model = model_cls_wrapper(T5ForConditionalGeneration).from_pretrained( File "seq2seq/eval_run_seq2seq.py", line 184, in model_cls=model_cls, picard_args=picard_args, tokenizer=tokenizer, schemas=dataset_splits.schemas File "/app/seq2seq/utils/custom_picard_model_wrapper.py", line 482, in with_picard asyncio.run(_init_picard(), debug=False) File "/opt/conda/lib/python3.7/asyncio/runners.py", line 43, in run return loop.run_until_complete(main) File "/opt/conda/lib/python3.7/asyncio/base_events.py", line 587, in run_until_complete return future.result() File "/app/seq2seq/utils/custom_picard_model_wrapper.py", line 126, in _init_picard await _register_schema(db_id=db_id, db_info=db_info, picard_client=client) File "/app/seq2seq/utils/custom_picard_model_wrapper.py", line 132, in _register_schema await picard_client.registerSQLSchema(db_id, sql_schema) thrift.py3.exceptions.TransportError: (<TransportErrorType.UNKNOWN: 0>, 'Channel is !good()', 0, <TransportOptions.0: 0>)

`

Enviornment : Macbook

Tomcatiiii commented 1 year ago

Hello, I fixed this problem by changing time.sleep(1) to 10 in line seq2seq/utils/picard_model_wrapper.py 95. I guess because the child process has not started successfully, the main process connected, so this error will be reported.

Rautmare-Sneha commented 1 year ago

Thank you @Tomcatiiii for the suggestion. I updated code as per your comment, but still facing same error. Is there any other way to resolve it. I am trying to reproduce the results of RASAT+PICARD, and facing this issue.