search

google / sling

SLING - A natural language frame semantics parser
Apache License 2.0
1.93k stars 268 forks source link

[Crash Report] Python pre-built wheels crash while loading .flow models #269

Closed creatorrr closed 6 years ago

creatorrr commented 6 years ago

~This happens for both the 1.0.0 and the 2.0.0-caspar wheels on my Ubuntu 18.04 machine.~ Correction: This happens for only for the 2.0.0-caspar wheel on my Ubuntu 18.04 machine.

~The following same exception is raised in both cases:~ The following exception is raised:

[2018-10-03 21:07:42.345517: E sling/myelin/compute.cc:1140] No kernel supports lr_lstm/punctuation/Lookup of type Lookup
[2018-10-03 21:07:42.345560: F sling/myelin/compiler.cc:127] Check failed: net->Compile(*flow, library_)
[1]    5145 abort (core dumped)  python

Code to reproduce:


Python 2.7.15rc1 (default, Apr 15 2018, 21:51:34)                                                                                                                                                                    
[GCC 7.3.0] on linux2                                                                                                                                                                                                
Type "help", "copyright", "credits" or "license" for more information.                                                                                                                                               
>>> import sling                                                                                                                                                                                                     
>>> parser = sling.Parser("caspar.flow")```
creatorrr commented 6 years ago

Haven't tried a custom build yet but will post results of that here once I get a chance.

creatorrr commented 6 years ago

Same result for custom build. Could I be missing a dependency? Followed instructions in the caspar branch found here.

ringgaard commented 6 years ago

As stated in the document, the caspar.flow model "is not yet available" :) We are working on getting the SLING 2.0 release ready (aka caspar), and it seems like the current caspar.model is out of sync with the code base.

The specific problem is that new models no longer depend on the the DRAGNN kernels so we removed the dependency in the parser to //sling/myelin/kernel:dragnn (see #262). However, the caspar.model you have downloaded still uses some of these kernels. If you want to use the parser with this model you will have to revert the changes to sling/nlp/parser/parser.cc and sling/nlp/parser/BUILD.

creatorrr commented 6 years ago

Ah, gotcha. Thanks @ringgaard! Closing this issue now since it'll be resolved when the 2.0 release happens.