veronica320
Different instance type in srl.py from utils and the SRL output
I know that the SRL tags are different in the current version of the SRL software employed. However, I noticed that besides this detail, it seems that there is another discrepancy between versions. The output of the SRL seems not to produce the sent_id field, nonetheless this is a required field for the Zeroshot-Event-Extraction (line 26 of srl.py module). Is there a way around this issue?
For now, I am considering to modify again the SRL code to copy this field. But this solution looks cumbersome. The error that raised is the following if I don't modify nothing regarding this issue:
File "source/predict_evaluate.py", line 62, in <module> model = EventDetector(config) File "/home/user/zero-shot-participant/Zeroshot-Event-Extraction/source/model.py", line 88, in __init__ self.verb_srl_dict, self.nom_srl_dict = srl.load_srl(input_file) File "/home/user/zero-shot-participant/Zeroshot-Event-Extraction/source/utils/srl.py", line 26, in load_srl sent_id = srl_res["sent_id"] KeyError: 'sent_id'
Hello, thanks for your interest, and sorry for the delayed reply due to holiday travels.
As mentioned in the README, the SRL model is not developed by the authors of this paper, and it has gone through several updates since then. You'll need to raise the issue under their repository instead. As a workaround, we can share the intermediate output from the SRL model on ACE if you'd like. Please shoot me an email and attach a copy of the ACE dataset you have (since it's propriety data, we'll need to know that you already have it) and I'm happy to share the output.
Sorry for the inconvenience!
Thanks for the reply. I made some changes to SRL code. Since I am trying to run your code on top of a different dataset, I need to go through all the process.
As soon as I can, I will made a fork in the SRL with my changes, in case anyone might be interested.